Skip to content
  1. Sep 14, 2012
    • Chandler Carruth's avatar
      Remove some dead, commented out code Duncan spotted in review. · 796de484
      Chandler Carruth authored
      llvm-svn: 163889
      796de484
    • Chandler Carruth's avatar
    • Chandler Carruth's avatar
      Lots of comment fixes and cleanups from Duncan's review. · 93a21e7a
      Chandler Carruth authored
      llvm-svn: 163887
      93a21e7a
    • NAKAMURA Takumi's avatar
      SROA.cpp: Unbreak gcc, sorry! · 4bbca0bb
      NAKAMURA Takumi authored
      llvm-svn: 163886
      4bbca0bb
    • NAKAMURA Takumi's avatar
      f4619d16
    • Chandler Carruth's avatar
      Speculative change to try to fix older GCC versions that can't handle · 9a447db9
      Chandler Carruth authored
      the injected class name of a dependent base class here.
      
      llvm-svn: 163884
      9a447db9
    • Chandler Carruth's avatar
      Introduce a new SROA implementation. · 1b398ae0
      Chandler Carruth authored
      This is essentially a ground up re-think of the SROA pass in LLVM. It
      was initially inspired by a few problems with the existing pass:
      - It is subject to the bane of my existence in optimizations: arbitrary
        thresholds.
      - It is overly conservative about which constructs can be split and
        promoted.
      - The vector value replacement aspect is separated from the splitting
        logic, missing many opportunities where splitting and vector value
        formation can work together.
      - The splitting is entirely based around the underlying type of the
        alloca, despite this type often having little to do with the reality
        of how that memory is used. This is especially prevelant with unions
        and base classes where we tail-pack derived members.
      - When splitting fails (often due to the thresholds), the vector value
        replacement (again because it is separate) can kick in for
        preposterous cases where we simply should have split the value. This
        results in forming i1024 and i2048 integer "bit vectors" that
        tremendously slow down subsequnet IR optimizations (due to large
        APInts) and impede the backend's lowering.
      
      The new design takes an approach that fundamentally is not susceptible
      to many of these problems. It is the result of a discusison between
      myself and Duncan Sands over IRC about how to premptively avoid these
      types of problems and how to do SROA in a more principled way. Since
      then, it has evolved and grown, but this remains an important aspect: it
      fixes real world problems with the SROA process today.
      
      First, the transform of SROA actually has little to do with replacement.
      It has more to do with splitting. The goal is to take an aggregate
      alloca and form a composition of scalar allocas which can replace it and
      will be most suitable to the eventual replacement by scalar SSA values.
      The actual replacement is performed by mem2reg (and in the future
      SSAUpdater).
      
      The splitting is divided into four phases. The first phase is an
      analysis of the uses of the alloca. This phase recursively walks uses,
      building up a dense datastructure representing the ranges of the
      alloca's memory actually used and checking for uses which inhibit any
      aspects of the transform such as the escape of a pointer.
      
      Once we have a mapping of the ranges of the alloca used by individual
      operations, we compute a partitioning of the used ranges. Some uses are
      inherently splittable (such as memcpy and memset), while scalar uses are
      not splittable. The goal is to build a partitioning that has the minimum
      number of splits while placing each unsplittable use in its own
      partition. Overlapping unsplittable uses belong to the same partition.
      This is the target split of the aggregate alloca, and it maximizes the
      number of scalar accesses which become accesses to their own alloca and
      candidates for promotion.
      
      Third, we re-walk the uses of the alloca and assign each specific memory
      access to all the partitions touched so that we have dense use-lists for
      each partition.
      
      Finally, we build a new, smaller alloca for each partition and rewrite
      each use of that partition to use the new alloca. During this phase the
      pass will also work very hard to transform uses of an alloca into a form
      suitable for promotion, including forming vector operations, speculating
      loads throguh PHI nodes and selects, etc.
      
      After splitting is complete, each newly refined alloca that is
      a candidate for promotion to a scalar SSA value is run through mem2reg.
      
      There are lots of reasonably detailed comments in the source code about
      the design and algorithms, and I'm going to be trying to improve them in
      subsequent commits to ensure this is well documented, as the new pass is
      in many ways more complex than the old one.
      
      Some of this is still a WIP, but the current state is reasonbly stable.
      It has passed bootstrap, the nightly test suite, and Duncan has run it
      successfully through the ACATS and DragonEgg test suites. That said, it
      remains behind a default-off flag until the last few pieces are in
      place, and full testing can be done.
      
      Specific areas I'm looking at next:
      - Improved comments and some code cleanup from reviews.
      - SSAUpdater and enabling this pass inside the CGSCC pass manager.
      - Some datastructure tuning and compile-time measurements.
      - More aggressive FCA splitting and vector formation.
      
      Many thanks to Duncan Sands for the thorough final review, as well as
      Benjamin Kramer for lots of review during the process of writing this
      pass, and Daniel Berlin for reviewing the data structures and algorithms
      and general theory of the pass. Also, several other people on IRC, over
      lunch tables, etc for lots of feedback and advice.
      
      llvm-svn: 163883
      1b398ae0
  2. Sep 13, 2012
  3. Sep 12, 2012
  4. Sep 11, 2012
  5. Sep 10, 2012
  6. Sep 09, 2012
  7. Sep 08, 2012
  8. Sep 07, 2012
  9. Sep 06, 2012
  10. Sep 05, 2012
  11. Sep 04, 2012
    • Jakub Staszak's avatar
      Fix my previous patch (r163164). It does now what it is supposed to do: · 85a77875
      Jakub Staszak authored
      Doesn't set MadeChange to TRUE if BypassSlowDivision doesn't change anything.
      
      llvm-svn: 163165
      85a77875
    • Jakub Staszak's avatar
      Return false if BypassSlowDivision doesn't change anything. · 46beca63
      Jakub Staszak authored
      Also a few minor changes:
      - use pre-inc instead of post-inc
      - use isa instead of dyn_cast
      - 80 col
      - trailing spaces
      
      llvm-svn: 163164
      46beca63
    • Preston Gurd's avatar
      Generic Bypass Slow Div · cdf540d5
      Preston Gurd authored
      - CodeGenPrepare pass for identifying div/rem ops
      - Backend specifies the type mapping using addBypassSlowDivType
      - Enabled only for Intel Atom with O2 32-bit -> 8-bit
      - Replace IDIV with instructions which test its value and use DIVB if the value
      is positive and less than 256.
      - In the case when the quotient and remainder of a divide are used a DIV
      and a REM instruction will be present in the IR. In the non-Atom case
      they are both lowered to IDIVs and CSE removes the redundant IDIV instruction,
      using the quotient and remainder from the first IDIV. However,
      due to this optimization CSE is not able to eliminate redundant
      IDIV instructions because they are located in different basic blocks.
      This is overcome by calculating both the quotient (DIV) and remainder (REM)
      in each basic block that is inserted by the optimization and reusing the result
      values when a subsequent DIV or REM instruction uses the same operands.
      - Test cases check for the presents of the optimization when calculating
      either the quotient, remainder,  or both.
      
      Patch by Tyler Nowicki!
      
      llvm-svn: 163150
      cdf540d5
    • Nadav Rotem's avatar
      LICM may hoist an instruction with undefined behavior above a trap. · 03dcd85b
      Nadav Rotem authored
      Scan the body of the loop and find instructions that may trap.
      Use this information when deciding if it is safe to hoist or sink instructions.
      Notice that we can optimize the search of instructions that may throw in the case of nested loops.
      
      rdar://11518836
      
      llvm-svn: 163132
      03dcd85b
  12. Sep 02, 2012
Loading