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

[Jim Grosbach]

Author: grosbach
Date: Wed Oct 13 17:55:33 2010
New Revision: 116449

URL: http://llvm.org/viewvc/llvm-project?rev=116449&view=rev
Log:
Add a 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=116449&r1=116448&r2=116449&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMInstrInfo.td (original)
+++ llvm/trunk/lib/Target/ARM/ARMInstrInfo.td Wed Oct 13 17:55:33 2010
@@ -953,6 +953,11 @@
 }
 
 // Address computation and loads and stores in PIC mode.
+// FIXME: These PIC insn patterns are pseudos, but derive from the normal insn
+//        classes (AXI1, et.al.) and so have encoding information and such,
+//        which is suboptimal. Once the rest of the code emitter (including
+//        JIT) is MC-ized we should look at refactoring these into true
+//        pseudos.
 let isNotDuplicable = 1 in {
 def PICADD : AXI1<0b0100, (outs GPR:$dst), (ins GPR:$a, pclabel:$cp, pred:$p),
                   Pseudo, IIC_iALUr, "",