[llvm-commits] [llvm] r82948 - in /llvm/trunk: lib/Target/ARM/ARMISelDAGToDAG.cpp lib/Target/ARM/ARMInstrInfo.cpp lib/Target/ARM/ARMInstrInfo.td lib/Target/ARM/ARMInstrThumb2.td test/CodeGen/ARM/t2-imm.ll

[Anton Korobeynikov]

Hello, Evan

> Hmm. This is interesting. Since MOVTi16 has a register source operand,
> it's not safe for it to be rematerialized. This change is wrong, but
> it's not going to be exercised.
Oh, right. I've disabled remtting in r82975 for now

> That said, I think it's good idea to re-materialize these
> instructions. So I believe the correct solution is to add a pseudo
> instruction that's printed as a pair of movw + movt instructions. That
> allows it to be re-materializable.
Sounds good.

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University