[llvm-commits] [llvm] r120447 - /llvm/trunk/lib/Target/ARM/ARMFastISel.cpp

[Eric Christopher]

Author: echristo
Date: Tue Nov 30 13:14:07 2010
New Revision: 120447

URL: http://llvm.org/viewvc/llvm-project?rev=120447&view=rev
Log:
Noticed this on inspection, fix and update some comments.

Modified:
    llvm/trunk/lib/Target/ARM/ARMFastISel.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMFastISel.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMFastISel.cpp?rev=120447&r1=120446&r2=120447&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMFastISel.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMFastISel.cpp Tue Nov 30 13:14:07 2010
@@ -933,12 +933,13 @@
                                   MachineMemOperand::MOLoad,
                                   MFI.getObjectSize(FI),
                                   MFI.getObjectAlignment(FI));
-    // LDRH needs an additional operand.
+    // ARM::STRH needs an additional operand.
     if (!isThumb && VT.getSimpleVT().SimpleTy == MVT::i16)
       AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DL,
                               TII.get(StrOpc))
                       .addReg(SrcReg, getKillRegState(true))
-                      .addFrameIndex(FI).addImm(Offset).addMemOperand(MMO));
+                      .addFrameIndex(FI).addReg(0).addImm(Offset)
+		      .addMemOperand(MMO));
     else
       AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DL,
                               TII.get(StrOpc))
@@ -948,7 +949,7 @@
     return true;
   }
 
-  // ARM::LDRH needs an additional operand.
+  // ARM::STRH needs an additional operand.
   if (!isThumb && VT.getSimpleVT().SimpleTy == MVT::i16)
     AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DL,
                             TII.get(StrOpc))