Elaborate on FIXME.
authorJim Grosbach <grosbach@apple.com>
Wed, 1 Dec 2010 04:01:17 +0000 (04:01 +0000)
committerJim Grosbach <grosbach@apple.com>
Wed, 1 Dec 2010 04:01:17 +0000 (04:01 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@120552 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/ARM/ARMInstrInfo.td

index 04a2d48310beb775b9067f584c96645c5acca497..9b291dfdb4ccc07124af4c45065de45b05a21fc8 100644 (file)
@@ -1179,7 +1179,10 @@ def PICSTRB : ARMPseudoInst<(outs), (ins GPR:$src, addrmodepc:$addr, pred:$p),
 // 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", []>;