Skip to content
  1. Sep 16, 2012
  2. Sep 15, 2012
    • Benjamin Kramer's avatar
      X86: Emitting x87 fsin/fcos for sinf/cosf is not safe without unsafe fp math. · ece43425
      Benjamin Kramer authored
      This was only an issue if sse is disabled.
      
      llvm-svn: 163967
      ece43425
    • Chandler Carruth's avatar
      Port the SSAUpdater-based promotion logic from the old SROA pass to the · 70b44c5c
      Chandler Carruth authored
      new one, and add support for running the new pass in that mode and in
      that slot of the pass manager. With this the new pass can completely
      replace the old one within the pipeline.
      
      The strategy for enabling or disabling the SSAUpdater logic is to do it
      by making the requirement of the domtree analysis optional. By default,
      it is required and we get the standard mem2reg approach. This is usually
      the desired strategy when run in stand-alone situations. Within the
      CGSCC pass manager, we disable requiring of the domtree analysis and
      consequentially trigger fallback to the SSAUpdater promotion.
      
      In theory this would allow the pass to re-use a domtree if one happened
      to be available even when run in a mode that doesn't require it. In
      practice, it lets us have a single pass rather than two which was
      simpler for me to wrap my head around.
      
      There is a hidden flag to force the use of the SSAUpdater code path for
      the purpose of testing. The primary testing strategy is just to run the
      existing tests through that path. One notable difference is that it has
      custom code to handle lifetime markers, and one of the tests has been
      enhanced to exercise that code.
      
      This has survived a bootstrap and the test suite without serious
      correctness issues, however my run of the test suite produced *very*
      alarming performance numbers. I don't entirely understand or trust them
      though, so more investigation is on-going.
      
      To aid my understanding of the performance impact of the new SROA now
      that it runs throughout the optimization pipeline, I'm enabling it by
      default in this commit, and will disable it again once the LNT bots have
      picked up one iteration with it. I want to get those bots (which are
      much more stable) to evaluate the impact of the change before I jump to
      any conclusions.
      
      NOTE: Several Clang tests will fail because they run -O3 and check the
      result's order of output. They'll go back to passing once I disable it
      again.
      
      llvm-svn: 163965
      70b44c5c
    • Akira Hatanaka's avatar
      Handled unaligned load/stores properly in Mips16 · 189d0add
      Akira Hatanaka authored
      Patch by Reed Kotler.
      
      llvm-svn: 163956
      189d0add
    • Manman Ren's avatar
      PGO: preserve branch-weight metadata when simplifying two branches with a common · bfb9d435
      Manman Ren authored
      destination.
      
      Updated previous implementation to fix a case not covered:
      // PBI: br i1 %x, TrueDest, BB
      // BI:  br i1 %y, TrueDest, FalseDest
      The other case was handled correctly.
      // PBI: br i1 %x, BB, FalseDest
      // BI:  br i1 %y, TrueDest, FalseDest
      
      Also tried to use 64-bit arithmetic instead of APInt with scale to simplify the
      computation. Let me know if you have other opinions about this.
      
      llvm-svn: 163954
      bfb9d435
  3. Sep 14, 2012
    • Manman Ren's avatar
      PGO: preserve branch-weight metadata when simplifying a switch with a single · 8691e522
      Manman Ren authored
      case to a conditional branch and when removing dead cases.
      
      llvm-svn: 163942
      8691e522
    • Alex Rosenberg's avatar
      Review feedback from Duncan Sands. Alphabetize includes and simplify · af2808cb
      Alex Rosenberg authored
      lit config.
      
      llvm-svn: 163928
      af2808cb
    • Manman Ren's avatar
      PGO: preserve branch-weight metadata when merging two switches where · d81b8e88
      Manman Ren authored
      the default target of the first switch is not the basic block the second switch
      is in (PredDefault != BB).
      
      llvm-svn: 163916
      d81b8e88
    • 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
    • Eric Christopher's avatar
      Fix both the test for zero and what we do if we have a zero for · b83dba2b
      Eric Christopher authored
      umulo legalization.
      
      Fixes PR13839
      
      llvm-svn: 163856
      b83dba2b
    • Jim Grosbach's avatar
      Assembler: Darwin variables defined via .set are no-dead-strip. · b7b750d4
      Jim Grosbach authored
      For gas compatibility.
      
      rdar://12219394
      
      llvm-svn: 163854
      b7b750d4
  4. Sep 13, 2012
  5. Sep 12, 2012
    • Michael Liao's avatar
      Fix PR11985 · abb87d48
      Michael Liao authored
          
      - BlockAddress has no support of BA + offset form and there is no way to
        propagate that offset into machine operand;
      - Add BA + offset support and a new interface 'getTargetBlockAddress' to
        simplify target block address forming;
      - All targets are modified to use new interface and X86 backend is enhanced to
        support BA + offset addressing.
      
      llvm-svn: 163743
      abb87d48
    • Dan Gohman's avatar
      Detect overflow in the path count computation. rdar://12277446. · 7c84dad8
      Dan Gohman authored
      llvm-svn: 163739
      7c84dad8
    • Eric Christopher's avatar
      Revert "Add some support for dealing with an object pointer on arguments." · c44e973a
      Eric Christopher authored
      This should be done on the subprogram, not the variable itself.
      
      llvm-svn: 163734
      c44e973a
    • Manman Ren's avatar
      PGO: preserve branch-weight metadata when removing a case which jumps · 49dbe255
      Manman Ren authored
      to the default target.
      
      llvm-svn: 163724
      49dbe255
    • Roman Divacky's avatar
      This patch corrects logic in PPCFrameLowering for save and restore of ... · c9e23d93
      Roman Divacky authored
      This patch corrects logic in PPCFrameLowering for save and restore of                                              
      nonvolatile condition register fields across calls under the SVR4 ABIs.                                            
                                                                                                                         
       * With the 64-bit ABI, the save location is at a fixed offset of 8 from                                           
      the stack pointer.  The frame pointer cannot be used to access this                                                
      portion of the stack frame since the distance from the frame pointer may                                           
      change with alloca calls.                                                                                          
                                                                                                                         
       * With the 32-bit ABI, the save location is just below the general
      register save area, and is accessed via the frame pointer like the rest
      of the save areas.  This is an optional slot, so it must only be created                                           
      if any of CR2, CR3, and CR4 were modified.                                                                      
                                                                                                                         
       * For both ABIs, save/restore logic is generated only if one of the     
      nonvolatile CR fields were modified.                                   
      
      I also took this opportunity to clean up an extra FIXME in
      PPCFrameLowering.h.  Save area offsets for 32-bit GPRs are meaningless
      for the 64-bit ABI, so I removed them for correctness and efficiency.
      
      
      Fixes PR13708 and partially also PR13623. It lets us enable exception handling
      on PPC64.
      
      Patch by William J. Schmidt!
      
      llvm-svn: 163713
      c9e23d93
    • Kristof Beyls's avatar
      Fix constant folding through bitcasts by no longer relying on undefined... · e6b876f4
      Kristof Beyls authored
      Fix constant folding through bitcasts by no longer relying on undefined behaviour (converting NaN values between float and double).
      
      SelectionDAG::getConstantFP(double Val, EVT VT, bool isTarget);
      should not be used when Val is not a simple constant (as the comment in
      SelectionDAG.h indicates). This patch avoids using this function
      when folding an unknown constant through a bitcast, where it cannot be
      guaranteed that Val will be a simple constant.
      
      llvm-svn: 163703
      e6b876f4
    • Nadav Rotem's avatar
      Stack coloring: remove lifetime intervals which contain escaped allocas. · 8ff00989
      Nadav Rotem authored
      The input program may contain intructions which are not inside lifetime
      markers. This can happen due to a bug in the compiler or due to a bug in
      user code (for example, returning a reference to a local variable).
      This commit adds checks that all of the instructions in the function and
      invalidates lifetime ranges which do not contain all of the instructions.
      
      llvm-svn: 163678
      8ff00989
    • Eric Christopher's avatar
      97c0fdd1
  6. Sep 11, 2012
  7. Sep 10, 2012
Loading