Skip to content
  1. Mar 29, 2010
  2. Mar 28, 2010
  3. Mar 27, 2010
  4. Mar 26, 2010
  5. Mar 25, 2010
    • Evan Cheng's avatar
      Code clean up. · 510bda20
      Evan Cheng authored
      llvm-svn: 99544
      510bda20
    • Chris Lattner's avatar
      fix a valgrind error on copy-constructor-synthesis.cpp, which is caused when · fc4ec253
      Chris Lattner authored
      the custom insertion hook deletes the instruction, then we try to set dead
      flags on it.  Neither the code that I added nor the code that was there 
      before was safe.
      
      llvm-svn: 99538
      fc4ec253
    • Evan Cheng's avatar
      Remove an unused option. · a1d0a027
      Evan Cheng authored
      llvm-svn: 99537
      a1d0a027
    • Devang Patel's avatar
      Add comment. · 95cd4b9c
      Devang Patel authored
      llvm-svn: 99507
      95cd4b9c
    • Evan Cheng's avatar
      Scheduler assumes SDDbgValue nodes are in source order. That's true currently.... · 1889440b
      Evan Cheng authored
      Scheduler assumes SDDbgValue nodes are in source order. That's true currently. But add an assertion to verify it.
      
      llvm-svn: 99501
      1889440b
    • Chris Lattner's avatar
      Change tblgen to emit FOOISD opcode names as two · 552dddc5
      Chris Lattner authored
      bytes instead of one byte.  This is important because
      we're running up to too many opcodes to fit in a byte
      and it is aggrevated by FIRST_TARGET_MEMORY_OPCODE
      making the numbering sparse.  This just bites the
      bullet and bloats out the table.  In practice, this
      increases the size of the x86 isel table from 74.5K
      to 76K.  I think we'll cope :)
      
      This fixes rdar://7791648
      
      llvm-svn: 99494
      552dddc5
    • Evan Cheng's avatar
      Remove a fixme that doesn't make sense any more. · 08b3364c
      Evan Cheng authored
      llvm-svn: 99489
      08b3364c
    • Evan Cheng's avatar
      7f0b16a2
    • Chris Lattner's avatar
      Make the NDEBUG assertion stronger and more clear what is · 4690af85
      Chris Lattner authored
      happening.
      
      Enhance scheduling to set the DEAD flag on implicit defs
      more aggressively.  Before, we'd set an implicit def operand
      to dead if it were present in the SDNode corresponding to
      the machineinstr but had no use.  Now we do it in this case
      AND if the implicit def does not exist in the SDNode at all.
      
      This exposes a couple of problems: one is the FIXME, which
      causes a live intervals crash on CodeGen/X86/sibcall.ll.
      The second is that it makes machinecse and licm more 
      aggressive (which is a good thing) but also exposes a case
      where licm hoists a set0 and then it doesn't get resunk.
      
      Talking to codegen folks about both these issues, but I need
      this patch in in the meantime.
      
      llvm-svn: 99485
      4690af85
    • Chris Lattner's avatar
      reapply 99444/99445, which I speculatively reverted in · e2a504ee
      Chris Lattner authored
      r99453.
      
      llvm-svn: 99482
      e2a504ee
    • Evan Cheng's avatar
      Change how dbg_value sdnodes are converted into machine instructions. Their... · 563fe3cc
      Evan Cheng authored
      Change how dbg_value sdnodes are converted into machine instructions. Their placement should be determined by the relative order of incoming llvm instructions. The scheduler will now use the SDNode ordering information to determine where to insert them. A dbg_value instruction is inserted after the instruction with the last highest source order and before the instruction with the next highest source order. It will optimize the placement by inserting right after the instruction that produces the value if they have consecutive order numbers.
      
      Here is a theoretical example that illustrates why the placement is important.
      
      tmp1 = 
      store tmp1 -> x
      ...
      tmp2 = add ...
      ...
      call
      ...
      store tmp2 -> x
      
      Now mem2reg comes along:
      
      tmp1 = 
      dbg_value (tmp1 -> x)
      ...
      tmp2 = add ...
      ...
      call
      ...
      dbg_value (tmp2 -> x)
      
      When the debugger examine the value of x after the add instruction but before the call, it should have the value of tmp1.
      
      Furthermore, for dbg_value's that reference constants, they should not be emitted at the beginning of the block (since they do not have "producers").
      
      This patch also cleans up how SDISel manages DbgValue nodes. It allow a SDNode to be referenced by multiple SDDbgValue nodes. When a SDNode is deleted, it uses the information to find the SDDbgValues and invalidate them. They are not deleted until the corresponding SelectionDAG is destroyed.
      
      llvm-svn: 99469
      563fe3cc
    • Evan Cheng's avatar
      Avoid being influenced by dbg_value instructions. · eae56b57
      Evan Cheng authored
      llvm-svn: 99465
      eae56b57
    • Chris Lattner's avatar
      revert 99444/99445. This doesn't cause the failure of · ddca7b09
      Chris Lattner authored
      2006-07-19-stwbrx-crash.ll for me, but it's the only likely
      patch in the blame list of several bots.  Lets see if this
      fixes it.
      
      llvm-svn: 99453
      ddca7b09
  6. Mar 24, 2010
Loading