Skip to content
  1. Nov 27, 2012
    • Chad Rosier's avatar
      [arm fast-isel] Appease the machine verifier by using the proper register · b4ac423e
      Chad Rosier authored
      classes.  The vast majority of the remaining issues are due to uses of
      invalid registers, which are defined by getRegForValue().  Those will be
      a little more challenging to cleanup.
      rdar://12719844
      
      llvm-svn: 168735
      b4ac423e
    • Chad Rosier's avatar
      0c007580
    • Chad Rosier's avatar
      [arm fast-isel] Appease the machine verifier by using the proper register · 2ec7db09
      Chad Rosier authored
      classes.  Also a bit of cleanup.
      rdar://12719844
      
      llvm-svn: 168728
      2ec7db09
    • Andrew Kaylor's avatar
      Moving SectionMemoryManager into RuntimeDyld and adding unit tests for it. · ab5ba51a
      Andrew Kaylor authored
      The SectionMemoryManager now supports (and requires) applying section-specific page permissions.  Clients using this memory manager must call either MCJIT::finalizeObject() or SectionMemoryManager::applyPermissions() before executing JITed code.
      
      See r168718 for changes from the previous implementation.
      
      llvm-svn: 168721
      ab5ba51a
    • Preston Briggs's avatar
      Modified depends() to recognize that when all levels are "=" and · 5cb8cfae
      Preston Briggs authored
      there's no possible loo-independent dependence, then there's no
      dependence.
      
      Updated all test result appropriately.
      
      llvm-svn: 168719
      5cb8cfae
    • Manman Ren's avatar
      CSE: allow PerformTrivialCoalescing to check copies across basic block · f89406ac
      Manman Ren authored
      boundaries.
      
      Given the following case:
      BB0
        %vreg1<def> = SUBrr %vreg0, %vreg7
        %vreg2<def> = COPY %vreg7
      BB1
        %vreg10<def> = SUBrr %vreg0, %vreg2
      We should be able to CSE between SUBrr in BB0 and SUBrr in BB1.
      
      rdar://12462006
      
      llvm-svn: 168717
      f89406ac
    • Meador Inge's avatar
      instcombine: Don't replace all uses for instructions with no uses · f1bc9e74
      Meador Inge authored
      My commit to migrate the printf simplifiers from the simplify-libcalls
      in r168604 introduced a regression reported by Duncan [1].  The problem
      is that in some cases the library call simplifier can return a new value
      that has no uses and the new value's type is different than the old value's
      type (which is fine because there are no uses).  The specific case that
      triggered the bug looked something like:
      
         declare void @printf(i8*, ...)
         ...
         call void (i8*, ...)* @printf(i8* %fmt)
      
      Which we want to optimized into:
      
         call i32 @putchar(i32 104)
      
      However, the code was attempting to replace all uses of the printf with
      the putchar and the types differ, hence a crash.  This is fixed by *just*
      deleting the original instruction when there are no uses.  The old
      simplify-libcalls pass is already doing something similar.
      
      [1] http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-November/056338.html
      
      llvm-svn: 168716
      f1bc9e74
    • Jakub Staszak's avatar
      Remove duplicated #includes. · 38e2f52e
      Jakub Staszak authored
      llvm-svn: 168712
      38e2f52e
    • Benjamin Kramer's avatar
      SCEV: Even if the latch terminator is foldable we can't deduce the result of... · e20e1242
      Benjamin Kramer authored
      SCEV: Even if the latch terminator is foldable we can't deduce the result of an unrelated condition with it.
      
      Fixes PR14432.
      
      llvm-svn: 168711
      e20e1242
    • Manman Ren's avatar
      X86: do not fold load instructions such as [V]MOVS[S|D] to other instructions · 5b462820
      Manman Ren authored
      when the destination register is wider than the memory load.
      
      These load instructions load from m32 or m64 and set the upper bits to zero,
      while the folded instructions may accept m128.
      
      rdar://12721174
      
      llvm-svn: 168710
      5b462820
    • Bill Schmidt's avatar
      This patch implements medium code model support for 64-bit PowerPC. · 34627e34
      Bill Schmidt authored
      The default for 64-bit PowerPC is small code model, in which TOC entries
      must be addressable using a 16-bit offset from the TOC pointer.  Additionally,
      only TOC entries are addressed via the TOC pointer.
      
      With medium code model, TOC entries and data sections can all be addressed
      via the TOC pointer using a 32-bit offset.  Cooperation with the linker
      allows 16-bit offsets to be used when these are sufficient, reducing the
      number of extra instructions that need to be executed.  Medium code model
      also does not generate explicit TOC entries in ".section toc" for variables
      that are wholly internal to the compilation unit.
      
      Consider a load of an external 4-byte integer.  With small code model, the
      compiler generates:
      
      	ld 3, .LC1@toc(2)
      	lwz 4, 0(3)
      
      	.section	.toc,"aw",@progbits
      .LC1:
      	.tc ei[TC],ei
      
      With medium model, it instead generates:
      
      	addis 3, 2, .LC1@toc@ha
      	ld 3, .LC1@toc@l(3)
      	lwz 4, 0(3)
      
      	.section	.toc,"aw",@progbits
      .LC1:
      	.tc ei[TC],ei
      
      Here .LC1@toc@ha is a relocation requesting the upper 16 bits of the
      32-bit offset of ei's TOC entry from the TOC base pointer.  Similarly,
      .LC1@toc@l is a relocation requesting the lower 16 bits.  Note that if
      the linker determines that ei's TOC entry is within a 16-bit offset of
      the TOC base pointer, it will replace the "addis" with a "nop", and
      replace the "ld" with the identical "ld" instruction from the small
      code model example.
      
      Consider next a load of a function-scope static integer.  For small code
      model, the compiler generates:
      
      	ld 3, .LC1@toc(2)
      	lwz 4, 0(3)
      
      	.section	.toc,"aw",@progbits
      .LC1:
      	.tc test_fn_static.si[TC],test_fn_static.si
      	.type	test_fn_static.si,@object
      	.local	test_fn_static.si
      	.comm	test_fn_static.si,4,4
      
      For medium code model, the compiler generates:
      
      	addis 3, 2, test_fn_static.si@toc@ha
      	addi 3, 3, test_fn_static.si@toc@l
      	lwz 4, 0(3)
      
      	.type	test_fn_static.si,@object
      	.local	test_fn_static.si
      	.comm	test_fn_static.si,4,4
      
      Again, the linker may replace the "addis" with a "nop", calculating only
      a 16-bit offset when this is sufficient.
      
      Note that it would be more efficient for the compiler to generate:
      
      	addis 3, 2, test_fn_static.si@toc@ha
              lwz 4, test_fn_static.si@toc@l(3)
      
      The current patch does not perform this optimization yet.  This will be
      addressed as a peephole optimization in a later patch.
      
      For the moment, the default code model for 64-bit PowerPC will remain the
      small code model.  We plan to eventually change the default to medium code
      model, which matches current upstream GCC behavior.  Note that the different
      code models are ABI-compatible, so code compiled with different models will
      be linked and execute correctly.
      
      I've tested the regression suite and the application/benchmark test suite in
      two ways:  Once with the patch as submitted here, and once with additional
      logic to force medium code model as the default.  The tests all compile
      cleanly, with one exception.  The mandel-2 application test fails due to an
      unrelated ABI compatibility with passing complex numbers.  It just so happens
      that small code model was incredibly lucky, in that temporary values in 
      floating-point registers held the expected values needed by the external
      library routine that was called incorrectly.  My current thought is to correct
      the ABI problems with _Complex before making medium code model the default,
      to avoid introducing this "regression."
      
      Here are a few comments on how the patch works, since the selection code
      can be difficult to follow:
      
      The existing logic for small code model defines three pseudo-instructions:
      LDtoc for most uses, LDtocJTI for jump table addresses, and LDtocCPT for
      constant pool addresses.  These are expanded by SelectCodeCommon().  The
      pseudo-instruction approach doesn't work for medium code model, because
      we need to generate two instructions when we match the same pattern.
      Instead, new logic in PPCDAGToDAGISel::Select() intercepts the TOC_ENTRY
      node for medium code model, and generates an ADDIStocHA followed by either
      a LDtocL or an ADDItocL.  These new node types correspond naturally to
      the sequences described above.
      
      The addis/ld sequence is generated for the following cases:
       * Jump table addresses
       * Function addresses
       * External global variables
       * Tentative definitions of global variables (common linkage)
      
      The addis/addi sequence is generated for the following cases:
       * Constant pool entries
       * File-scope static global variables
       * Function-scope static variables
      
      Expanding to the two-instruction sequences at select time exposes the
      instructions to subsequent optimization, particularly scheduling.
      
      The rest of the processing occurs at assembly time, in
      PPCAsmPrinter::EmitInstruction.  Each of the instructions is converted to
      a "real" PowerPC instruction.  When a TOC entry needs to be created, this
      is done here in the same manner as for the existing LDtoc, LDtocJTI, and
      LDtocCPT pseudo-instructions (I factored out a new routine to handle this).
      
      I had originally thought that if a TOC entry was needed for LDtocL or
      ADDItocL, it would already have been generated for the previous ADDIStocHA.
      However, at higher optimization levels, the ADDIStocHA may appear in a 
      different block, which may be assembled textually following the block
      containing the LDtocL or ADDItocL.  So it is necessary to include the
      possibility of creating a new TOC entry for those two instructions.
      
      Note that for LDtocL, we generate a new form of LD called LDrs.  This
      allows specifying the @toc@l relocation for the offset field of the LD
      instruction (i.e., the offset is replaced by a SymbolLo relocation).
      When the peephole optimization described above is added, we will need
      to do similar things for all immediate-form load and store operations.
      
      The seven "mcm-n.ll" test cases are kept separate because otherwise the
      intermingling of various TOC entries and so forth makes the tests fragile
      and hard to understand.
      
      The above assumes use of an external assembler.  For use of the
      integrated assembler, new relocations are added and used by
      PPCELFObjectWriter.  Testing is done with "mcm-obj.ll", which tests for
      proper generation of the various relocations for the same sequences
      tested with the external assembler.
      
      llvm-svn: 168708
      34627e34
    • Ulrich Weigand's avatar
      Never use .lcomm on platforms where it does not accept an alignment · e5f94058
      Ulrich Weigand authored
      argument.  Instead, use a pair of .local and .comm directives.
      
      This avoids spurious differences between binaries built by the
      integrated assembler vs. those built by the external assembler,
      since the external assembler may impose alignment requirements
      on .lcomm symbols where the integrated assembler does not.
      
      llvm-svn: 168704
      e5f94058
    • Bill Wendling's avatar
      Remove the dependent libraries feature. · ee5984df
      Bill Wendling authored
      The dependent libraries feature was never used and has bit-rotted. Remove it.
      
      llvm-svn: 168694
      ee5984df
    • Dmitry Vyukov's avatar
      tsan: instrument atomic nand operation · a878e743
      Dmitry Vyukov authored
      llvm-svn: 168684
      a878e743
    • Eric Christopher's avatar
      Revert rearrangement of debug info sections to unblock the bots · 6e20a168
      Eric Christopher authored
      and O0 + debug codegen.
      
      llvm-svn: 168680
      6e20a168
    • Preston Briggs's avatar
      Modify depends(Src, Dst, PossiblyLoopIndependent). · 1084fa2e
      Preston Briggs authored
      If the Src and Dst are the same instruction,
      no loop-independent dependence is possible,
      so we force the PossiblyLoopIndependent flag to false.
      
      The test case results are updated appropriately.
      
      llvm-svn: 168678
      1084fa2e
    • Meador Inge's avatar
      instcombine: Migrate sprintf optimizations · 25c9b3b6
      Meador Inge authored
      This patch migrates the sprintf optimizations from the simplify-libcalls
      pass into the instcombine library call simplifier.
      
      llvm-svn: 168677
      25c9b3b6
    • Jakub Staszak's avatar
      Remove unneeded #include. · 8262b885
      Jakub Staszak authored
      llvm-svn: 168670
      8262b885
    • Eric Christopher's avatar
      The section is .debug_line. · 710c374d
      Eric Christopher authored
      llvm-svn: 168666
      710c374d
    • Jakub Staszak's avatar
      Remove unneeded #include. · 508888e4
      Jakub Staszak authored
      llvm-svn: 168664
      508888e4
    • NAKAMURA Takumi's avatar
      llvm/CodeGen: Remove empty files in r168659. · 2e4a3070
      NAKAMURA Takumi authored
      llvm-svn: 168663
      2e4a3070
    • Joe Abbey's avatar
      Code pretification · ac997e96
      Joe Abbey authored
      llvm-svn: 168661
      ac997e96
    • Jakub Staszak's avatar
      Remove unused forward declaration. · 08a28d24
      Jakub Staszak authored
      llvm-svn: 168660
      08a28d24
    • Jakub Staszak's avatar
      Remove unused MachineLoopRanges analysis. · 0820b2a3
      Jakub Staszak authored
      llvm-svn: 168659
      0820b2a3
    • Chad Rosier's avatar
      [arm fast-isel] Appease the machine verifier by using the proper register · 2e82ad12
      Chad Rosier authored
      classes.  The associated test case still doesn't pass, but it does have far
      fewer issues.
      rdar://12719844
      
      llvm-svn: 168657
      2e82ad12
    • Owen Anderson's avatar
      Revert r168635 "Step towards implementation of pass manager with... · 1db12f51
      Owen Anderson authored
      Revert r168635 "Step towards implementation of pass manager with doInitialization and doFinalization per module detangled from runOn?? calls, still has temporary code not to break ASAN to be removed when that pass conforms to the proposed model".
      It appears to have broken at least one buildbot.
      
      llvm-svn: 168654
      1db12f51
    • Michael Ilseman's avatar
      Fast-math optimization: fold multiply by zero · be9137a5
      Michael Ilseman authored
      Added in first optimization using fast-math flags to serve as an example for following optimizations. SimplifyInstruction will now try to optimize an fmul observing its FastMathFlags to see if it can fold multiply by zero when 'nnan' and 'nsz' flags are set.
      
      llvm-svn: 168648
      be9137a5
    • Michael Ilseman's avatar
      Fast-math flags for the bitcode · 9978d7e9
      Michael Ilseman authored
      Added in bitcode enum for the serializing of fast-math flags. Added in the reading/writing of fast-math flags from the OptimizationFlags record for BinaryOps.
      
      llvm-svn: 168646
      9978d7e9
    • Michael Ilseman's avatar
      Fast-math flags for LLVM IR parsing and printing · 9205317b
      Michael Ilseman authored
      Added in the ability to read LLVM IR text that contains fast-math flags as a sequence of capital letters separated by spaces in any order. Added in the printing of the fast-math flags in a canonical order, and don't print the other flags when 'fast' is specified, as 'fast' implies all the others.
      
      llvm-svn: 168645
      9205317b
    • Eric Christopher's avatar
      Make comment names match function names. · 69e328e5
      Eric Christopher authored
      llvm-svn: 168644
      69e328e5
    • Eric Christopher's avatar
      Add in sections for the fission case (no change so incorrect) and · 4c9b119d
      Eric Christopher authored
      add a TODO for starting.
      
      llvm-svn: 168643
      4c9b119d
    • Michael Ilseman's avatar
      Fast-math interfaces for Instructions · 149209ec
      Michael Ilseman authored
      Add in getter/setter methods for Instructions, allowing them to be the interface to FPMathOperator similarly to now NUS/NSW is handled.
      
      llvm-svn: 168642
      149209ec
    • Eric Christopher's avatar
      Reorder section output ordering. · c800b12b
      Eric Christopher authored
      llvm-svn: 168638
      c800b12b
    • Eric Christopher's avatar
      Whitespace cleanup. · 735401cf
      Eric Christopher authored
      llvm-svn: 168637
      735401cf
    • Owen Anderson's avatar
      Step towards implementation of pass manager with doInitialization and... · 336368c4
      Owen Anderson authored
      Step towards implementation of pass manager with doInitialization and doFinalization per module detangled from runOn?? calls, still has temporary code not to break ASAN to be removed when that pass conforms to the proposed model
      
      Patch by Pedro Artigas, with feedback from by Chandler Carruth.
      
      llvm-svn: 168635
      336368c4
    • Chad Rosier's avatar
      110b73e0
    • Chad Rosier's avatar
      Now that the X86 Maximal Stack Alignment Check pass has been removed (i.e., · f8a3a62c
      Chad Rosier authored
      r168627), we no longer need to call the freezeReservedRegs() function a second
      time.  Previously, this pass was conservatively adding the FP to the set of
      reserved registers, requiring the second update to the reserved registers.
      rdar://12719844
      
      llvm-svn: 168631
      f8a3a62c
    • Chad Rosier's avatar
      Now that the X86 Maximal Stack Alignment Check pass has been removed (i.e., · a44e1825
      Chad Rosier authored
      r168627), we no longer need to call the freezeReservedRegs() function a second
      time.  Previously, this pass was conservatively adding the FP to the set of
      reserved registers, requiring the second update to the reserved registers.
      rdar://12719844
      
      llvm-svn: 168630
      a44e1825
    • Eli Friedman's avatar
      Get rid of the getPointeeAlignment helper function from · b14873c4
      Eli Friedman authored
      InstCombineLoadStoreAlloca.cpp, which had many issues.
      (At least two bugs were noted on llvm-commits, and it was overly conservative.)
      Instead, use getOrEnforceKnownAlignment.
      
      llvm-svn: 168629
      b14873c4
  2. Nov 26, 2012
    • Chad Rosier's avatar
      Remove the X86 Maximal Stack Alignment Check pass as it is no longer necessary. · 4179e3f5
      Chad Rosier authored
      This pass was conservative in that it always reserved the FP to enable dynamic
      stack realignment, which allowed the RA to use aligned spills for vector
      registers.  This happens even when spills were not necessary.  The RA has 
      since been improved to use unaligned spills when necessary.
      
      The new behavior is to realign the stack if the frame pointer was already
      reserved for some other reason, but don't reserve the frame pointer just
      because a function contains vector virtual registers.
      
      Part of rdar://12719844
      
      llvm-svn: 168627
      4179e3f5
Loading