Skip to content
  1. Jul 08, 2010
  2. Jul 07, 2010
  3. Jul 03, 2010
  4. Jun 29, 2010
  5. Jun 23, 2010
  6. May 06, 2010
  7. Feb 09, 2010
  8. Jan 05, 2010
    • David Greene's avatar
      · c4878b13
      David Greene authored
      Change errs() to dbgs().
      
      llvm-svn: 92535
      c4878b13
  9. Dec 03, 2009
  10. Oct 25, 2009
  11. Oct 24, 2009
  12. Sep 28, 2009
  13. Sep 22, 2009
    • Evan Cheng's avatar
      Minor bug fix. LowerSubregs should translate · 8c500100
      Evan Cheng authored
      %S0<def> = EXTRACT_SUBREG %Q0<kill>, 1
      to
      %S0<def> = IMPLICIT_DEF %Q0<imp-use,kill>
      
      Implicit_def does not *read* any register so the operand should be marked "implicit". The missing "implicit" marker on the operand is wrong, but it doesn't actually break anything.
      
      llvm-svn: 82503
      8c500100
  14. Aug 22, 2009
  15. Aug 08, 2009
    • Jakob Stoklund Olesen's avatar
      Remove RegisterScavenger::isSuperRegUsed(). This completely reverses the mistaken commit r77904. · 8e96c6ab
      Jakob Stoklund Olesen authored
      Now there is no special treatment of instructions that redefine part of a
      super-register. Instead, the super-register is marked with <imp-use,kill> and
      <imp-def>. For instance, from LowerSubregs on ARM:
      
      subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1<undef>, %D1<kill>, 5
      subreg: %D2<def> = FCPYD %D1<kill>, 14, %reg0, %Q1<imp-def>
      
      subreg: CONVERTING: %Q1<def> = INSERT_SUBREG %Q1, %D0<kill>, 6
      subreg: %D3<def> = FCPYD %D0<kill>, 14, %reg0, %Q1<imp-use,kill>, %Q1<imp-def>
      llvm-svn: 78466
      8e96c6ab
  16. Aug 05, 2009
  17. Aug 04, 2009
    • Jakob Stoklund Olesen's avatar
      LowerSubregsInstructionPass::LowerExtract should not extend the live range of registers. · 6304369c
      Jakob Stoklund Olesen authored
      When LowerExtract eliminates an EXTRACT_SUBREG with a kill flag, it moves the
      kill flag to the place where the sub-register is killed. This can accidentally
      overlap with the use of a sibling sub-register, and we have trouble.
      
      In the test case we have this code:
      
      Live Ins: %R0 %R1 %R2
      	%R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
      	%R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
      	%R1L<def> = EXTRACT_SUBREG %R1<kill>, 1
      	%R0L<def> = EXTRACT_SUBREG %R0<kill>, 1
      	%R0H<def> = ADD16 %R2H<kill>, %R2L<kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
      
      subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
      subreg: eliminated!
      subreg: killed here: %R0H<def> = ADD16 %R2H, %R2L, %R2<imp-use,kill>, %AZ<imp-def>, %AN<imp-def>, %AC0<imp-def>, %V<imp-def>, %VS<imp-def>
      
      The kill flag on %R2 is moved to the last instruction, and the live range overlaps with the definition of %R2H:
      
      *** Bad machine code: Redefining a live physical register ***
      - function:    f
      - basic block:  0x18358c0 (#0)
      - instruction: %R2H<def> = LOAD16fi <fi#-1>, 0, Mem:LD(2,4) [FixedStack-1 + 0]
      Register R2H was defined but already live.
      
      The fix is to replace EXTRACT_SUBREG with IMPLICIT_DEF instead of eliminating
      it completely:
      
      subreg: CONVERTING: %R2L<def> = EXTRACT_SUBREG %R2<kill>, 1
      subreg: replace by: %R2L<def> = IMPLICIT_DEF %R2<kill>
      
      Note that these IMPLICIT_DEF instructions survive to the asm output. It is
      necessary to fix the stack-color-with-reg test case because of that.
      
      llvm-svn: 78093
      6304369c
  18. Aug 03, 2009
    • Jakob Stoklund Olesen's avatar
      Fix Bug 4657: register scavenger asserts with subreg lowering · 5d8ace09
      Jakob Stoklund Olesen authored
      When LowerSubregsInstructionPass::LowerInsert eliminates an INSERT_SUBREG
      instriction because it is an identity copy, make sure that the same registers
      are alive before and after the elimination.
      
      When the super-register is marked <undef> this requires inserting an
      IMPLICIT_DEF instruction to make sure the super register is live.
      
      Fix a related bug where a kill flag on the inserted sub-register was not transferred properly.
      
      Finally, clear the undef flag in MachineInstr::addRegisterKilled. Undef implies dead and kill implies live, so they cant both be valid.
      
      llvm-svn: 77989
      5d8ace09
  19. Aug 01, 2009
  20. Jul 25, 2009
    • Daniel Dunbar's avatar
      More migration to raw_ostream, the water has dried up around the iostream hole. · 0dd5e1ed
      Daniel Dunbar authored
       - Some clients which used DOUT have moved to DEBUG. We are deprecating the
         "magic" DOUT behavior which avoided calling printing functions when the
         statement was disabled. In addition to being unnecessary magic, it had the
         downside of leaving code in -Asserts builds, and of hiding potentially
         unnecessary computations.
      
      llvm-svn: 77019
      0dd5e1ed
  21. Jul 16, 2009
  22. Mar 23, 2009
    • Evan Cheng's avatar
      Do not fold away subreg_to_reg if the source register has a sub-register... · 47c9750f
      Evan Cheng authored
      Do not fold away subreg_to_reg if the source register has a sub-register index. That means the source register is taking a sub-register of a larger register. e.g. On x86
      %RAX<def> = ...
      %RAX<def> = SUBREG_TO_REG 0, %EAX:3<kill>, 3
      The first def is defining RAX, not EAX so the top bits were not zero-extended.
      
      llvm-svn: 67511
      47c9750f
  23. Dec 18, 2008
  24. Nov 21, 2008
  25. Oct 03, 2008
  26. Sep 25, 2008
  27. Sep 23, 2008
  28. Sep 22, 2008
  29. Sep 04, 2008
  30. Aug 20, 2008
  31. Aug 07, 2008
    • Dan Gohman's avatar
      Re-enable elimination of unnecessary SUBREG_TO_REG instructions in · 527ca7e2
      Dan Gohman authored
      LowerSubregs, and fix an x86-64 isel bug that this exposed.
      
      SUBREG_TO_REG for x86-64 implicit zero extension is only safe for
      isel to generate when the source is known to always have zeros in
      the high 32 bits. The EXTRACT_SUBREG instruction does not clear
      the high 32 bits.
      
      llvm-svn: 54444
      527ca7e2
Loading