[llvm-commits] [llvm] r82465 - in /llvm/trunk/lib/Target: SystemZ/SystemZInstrBuilder.h X86/X86InstrBuilder.h

Dan Gohman gohman at apple.com
Mon Sep 21 10:58:09 PDT 2009


Author: djg
Date: Mon Sep 21 12:58:09 2009
New Revision: 82465

URL: http://llvm.org/viewvc/llvm-project?rev=82465&view=rev
Log:
Fix the offset values for these memoperands. For frame objects, the
PseudoSourceValue already effectively represents the offset from the
frame base, so the actual offset should not be added to it.

Modified:
    llvm/trunk/lib/Target/SystemZ/SystemZInstrBuilder.h
    llvm/trunk/lib/Target/X86/X86InstrBuilder.h

Modified: llvm/trunk/lib/Target/SystemZ/SystemZInstrBuilder.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/SystemZ/SystemZInstrBuilder.h?rev=82465&r1=82464&r2=82465&view=diff

==============================================================================
--- llvm/trunk/lib/Target/SystemZ/SystemZInstrBuilder.h (original)
+++ llvm/trunk/lib/Target/SystemZ/SystemZInstrBuilder.h Mon Sep 21 12:58:09 2009
@@ -115,7 +115,7 @@
     Flags |= MachineMemOperand::MOStore;
   MachineMemOperand MMO(PseudoSourceValue::getFixedStack(FI),
                         Flags,
-                        MFI.getObjectOffset(FI) + Offset,
+                        Offset,
                         MFI.getObjectSize(FI),
                         MFI.getObjectAlignment(FI));
   return addOffset(MIB.addFrameIndex(FI), Offset)

Modified: llvm/trunk/lib/Target/X86/X86InstrBuilder.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrBuilder.h?rev=82465&r1=82464&r2=82465&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrBuilder.h (original)
+++ llvm/trunk/lib/Target/X86/X86InstrBuilder.h Mon Sep 21 12:58:09 2009
@@ -144,7 +144,7 @@
     Flags |= MachineMemOperand::MOStore;
   MachineMemOperand MMO(PseudoSourceValue::getFixedStack(FI),
                         Flags,
-                        MFI.getObjectOffset(FI) + Offset,
+                        Offset,
                         MFI.getObjectSize(FI),
                         MFI.getObjectAlignment(FI));
   return addOffset(MIB.addFrameIndex(FI), Offset)





More information about the llvm-commits mailing list