[llvm-commits] [llvm] r75048 - in /llvm/trunk/lib/Target/ARM: ARMAddressingModes.h ARMBaseInstrInfo.cpp ARMBaseRegisterInfo.cpp ARMCodeEmitter.cpp ARMISelDAGToDAG.cpp ARMInstrInfo.td ARMInstrThumb2.td ARMLoadStoreOptimizer.cpp AsmPrinter/ARMAsmPrinter.cpp

[Chris Lattner]

On Jul 8, 2009, at 2:04 PM, Evan Cheng wrote:

> Author: evancheng
> Date: Wed Jul  8 16:03:57 2009
> New Revision: 75048
>
> URL: http://llvm.org/viewvc/llvm-project?rev=75048&view=rev
> Log:
> Change how so_imm and t2_so_imm are handled. At instruction  
> selection time, the immediates are no longer encoded in the imm8 +  
> rot format, that are left as it is. The encoding is now done in ams  
> printing and code emission time instead.

Is this really needed?  Aren't the instructions that these end up in  
encoded very differently as well?  If so, why not just have them  
described "properly" when you pick a specific instruction?  How do you  
plan to have the JIT support these?

-Chris