[llvm-commits] [llvm] r120552 - /llvm/trunk/lib/Target/ARM/ARMInstrInfo.td

[Jim Grosbach]

Author: grosbach
Date: Tue Nov 30 22:01:17 2010
New Revision: 120552

URL: http://llvm.org/viewvc/llvm-project?rev=120552&view=rev
Log:
Elaborate on FIXME.

Modified:
    llvm/trunk/lib/Target/ARM/ARMInstrInfo.td

Modified: llvm/trunk/lib/Target/ARM/ARMInstrInfo.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMInstrInfo.td?rev=120552&r1=120551&r2=120552&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMInstrInfo.td (original)
+++ llvm/trunk/lib/Target/ARM/ARMInstrInfo.td Tue Nov 30 22:01:17 2010
@@ -1179,7 +1179,10 @@
 // assembler.
 let neverHasSideEffects = 1, isReMaterializable = 1 in
 // FIXME: We want one cannonical LEApcrel instruction and to express one or
-// both of these as pseudo-instructions that get expanded to it.
+// both of these as pseudo-instructions that get expanded to it. In particular,
+// the cannonical "adr" pattern should take a single label operand, and the
+// JT version should be a pseudo that when lowered to MC, xforms the insn
+// to the canonical form referencing the correct symbol.
 def LEApcrel : AXI1<0, (outs GPR:$Rd), (ins i32imm:$label, pred:$p),
                     MiscFrm, IIC_iALUi,
                     "adr${p}\t$Rd, #$label", []>;