Skip to content
  1. Sep 29, 2009
  2. Sep 28, 2009
  3. Sep 27, 2009
  4. Sep 26, 2009
  5. Sep 25, 2009
    • Evan Cheng's avatar
      Flip -disable-post-RA-scheduler to -post-RA-scheduler. · 3872b3c1
      Evan Cheng authored
      llvm-svn: 82803
      3872b3c1
    • Dan Gohman's avatar
      Improve MachineMemOperand handling. · 48b185d6
      Dan Gohman authored
       - Allocate MachineMemOperands and MachineMemOperand lists in MachineFunctions.
         This eliminates MachineInstr's std::list member and allows the data to be
         created by isel and live for the remainder of codegen, avoiding a lot of
         copying and unnecessary translation. This also shrinks MemSDNode.
       - Delete MemOperandSDNode. Introduce MachineSDNode which has dedicated
         fields for MachineMemOperands.
       - Change MemSDNode to have a MachineMemOperand member instead of its own
         fields with the same information. This introduces some redundancy, but
         it's more consistent with what MachineInstr will eventually want.
       - Ignore alignment when searching for redundant loads for CSE, but remember
         the greatest alignment.
      
      Target-specific code which previously used MemOperandSDNodes with generic
      SDNodes now use MemIntrinsicSDNodes, with opcodes in a designated range
      so that the SelectionDAG framework knows that MachineMemOperand information
      is available.
      
      llvm-svn: 82794
      48b185d6
    • Dan Gohman's avatar
      Rename getTargetNode to getMachineNode, for consistency with the · 32f71d71
      Dan Gohman authored
      naming scheme used in SelectionDAG, where there are multiple kinds
      of "target" nodes, but "machine" nodes are nodes which represent
      a MachineInstr.
      
      llvm-svn: 82790
      32f71d71
    • Dale Johannesen's avatar
      Make sure sin, cos, sqrt calls are marked readonly · a318d91a
      Dale Johannesen authored
      before producing FSIN, FCOS, FSQRT.  If they aren't
      so marked we have to assume they might set errno.
      
      llvm-svn: 82781
      a318d91a
    • Dale Johannesen's avatar
      Generate FSQRT from calls to the sqrt function, which · c7213426
      Dale Johannesen authored
      allows appropriate backends to generate a sqrt instruction.
      
      On x86, this isn't done at -O0 because we go through
      FastISel instead.  This is a behavior change from before
      this series of sqrt patches started.  I think this is OK
      considering that compile speed is most important at -O0, but
      could be convinced otherwise.
      
      llvm-svn: 82778
      c7213426
    • Bob Wilson's avatar
      pr4926: ARM requires the stack pointer to be aligned, even for leaf functions. · d60367c1
      Bob Wilson authored
      For the AAPCS ABI, SP must always be 4-byte aligned, and at any "public
      interface" it must be 8-byte aligned.  For the older ARM APCS ABI, the stack
      alignment is just always 4 bytes.  For X86, we currently align SP at
      entry to a function (e.g., to 16 bytes for Darwin), but no stack alignment
      is needed at other times, such as for a leaf function.
      
      After discussing this with Dan, I decided to go with the approach of adding
      a new "TransientStackAlignment" field to TargetFrameInfo.  This value
      specifies the stack alignment that must be maintained even in between calls.
      It defaults to 1 except for ARM, where it is 4.  (Some other targets may
      also want to set this if they have similar stack requirements. It's not
      currently required for PPC because it sets targetHandlesStackFrameRounding
      and handles the alignment in target-specific code.) The existing StackAlignment
      value specifies the alignment upon entry to a function, which is how we've
      been using it anyway.
      
      llvm-svn: 82767
      d60367c1
    • Nate Begeman's avatar
      Fix combiner-aa issue with bases which are different, but can alias. · 18150d5a
      Nate Begeman authored
      Previously, it treated GV+28 GV+0 as different bases, and assumed they could
      not alias.
      
      llvm-svn: 82753
      18150d5a
    • Dan Gohman's avatar
      Add a version of dumpr() that has a SelectionDAG* argument. · ebdfe4af
      Dan Gohman authored
      llvm-svn: 82742
      ebdfe4af
    • Jim Grosbach's avatar
      Start of revamping the register scavenging in PEI. ARM Thumb1 is the driving · 372e9a38
      Jim Grosbach authored
      interest for this, as it currently reserves a register rather than using
      the scavenger for matierializing constants as needed.
      
      Instead of scavenging registers on the fly while eliminating frame indices,
      new virtual registers are created, and then a scavenged collectively in a
      post-pass over the function. This isolates the bits that need to interact
      with the scavenger, and sets the stage for more intelligent use, and reuse,
      of scavenged registers.
      
      For the time being, this is disabled by default. Once the bugs are worked out,
      the current scavenging calls in replaceFrameIndices() will be removed and
      the post-pass scavenging will be the default. Until then,
      -enable-frame-index-scavenging enables the new code. Currently, only the
      Thumb1 back end is set up to use it.
      
      llvm-svn: 82734
      372e9a38
    • Mike Stump's avatar
      Delete space after function name, before (, reflow a comment and · 944fa259
      Mike Stump authored
      delete a few blank lines.
      
      llvm-svn: 82729
      944fa259
    • Mike Stump's avatar
      Fix spacing. · d6f9a2f9
      Mike Stump authored
      llvm-svn: 82727
      d6f9a2f9
  6. Sep 24, 2009
    • David Goodwin's avatar
      Make the end-of-itinerary mark explicit. Some cleanup. · bf97147a
      David Goodwin authored
      llvm-svn: 82709
      bf97147a
    • Bob Wilson's avatar
      Fix a hypothetical problem for targets with StackGrowsUp and a non-zero · 5fe313d6
      Bob Wilson authored
      LocalAreaOffset.  (We don't have any of those right now.)
      PEI::calculateFrameObjectOffsets includes the absolute value of the
      LocalAreaOffset in the cumulative offset value used to calculate the
      stack frame size.  It then adds the raw value of the LocalAreaOffset
      to the stack size.  For a StackGrowsDown target, that raw value is negative
      and has the effect of cancelling out the absolute value that was added
      earlier, but that obviously won't work for a StackGrowsUp target.  Change
      to subtract the absolute value of the LocalAreaOffset.
      
      llvm-svn: 82693
      5fe313d6
    • Chris Lattner's avatar
      unconditionally compute MMI even if the target doesn't support EH or Debug... · 87d8f2b9
      Chris Lattner authored
      unconditionally compute MMI even if the target doesn't support EH or Debug info, because the target may use it for other things, this fixes PR5036
      
      llvm-svn: 82684
      87d8f2b9
    • Evan Cheng's avatar
      Fix PR5024 with a big hammer: disable the double-def assertion in the scavenger. · 26ea28eb
      Evan Cheng authored
      LiveVariables add implicit kills to correctly track partial register kills. This works well enough and is fairly accurate. But coalescer can make it impossible to maintain these markers. e.g.
      
              BL <ga:sss1>, %R0<kill,undef>, %S0<kill>, %R0<imp-def>, %R1<imp-def,dead>, %R2<imp-def,dead>, %R3<imp-def,dead>, %R12<imp-def,dead>, %LR<imp-def,dead>, %D0<imp-def>, ...
      ...
      	%reg1031<def> = FLDS <cp#1>, 0, 14, %reg0, Mem:LD4[ConstantPool]
      ...
         	%S0<def> = FCPYS %reg1031<kill>, 14, %reg0, %D0<imp-use,kill>
      
      When reg1031 and S0 are coalesced, the copy (FCPYS) will be eliminated the the implicit-kill of D0 is lost. In this case it's possible to move the marker to the FLDS. But in many cases, this is not possible. Suppose
      
      	%reg1031<def> = FOO <cp#1>, %D0<imp-def>
      ...
         	%S0<def> = FCPYS %reg1031<kill>, 14, %reg0, %D0<imp-use,kill>
      
      When FCPYS goes away, the definition of S0 is the "FOO" instruction. However, transferring the D0 implicit-kill to FOO doesn't work since it is the def of D0 itself. We need to fix this in another time by introducing a "kill" pseudo instruction to track liveness.
      
      Disabling the assertion is not ideal, but machine verifier is doing that job now. It's important to know double-def is not a miscomputation since it means a register should be free but it's not tracked as free. It's a performance issue instead.
      
      llvm-svn: 82677
      26ea28eb
    • Evan Cheng's avatar
      Clean up LiveVariables and change how it deals with partial updates and kills.... · a21aac38
      Evan Cheng authored
      Clean up LiveVariables and change how it deals with partial updates and kills. This also eliminate the horrible check which scan forward to the end of the basic block. It should be faster and more accurate.
      
      llvm-svn: 82676
      a21aac38
  7. Sep 23, 2009
Loading