[PATCH] ARM: use new modified-imm instructions for all MC-level purposes.

Tim Northover t.p.northover at gmail.com
Tue Aug 27 08:52:06 PDT 2013


Hi Kristof,

> Wouldn’t it be possible to get rid of the “ri” MCInst versions quite easily,
> by adding a switch statement in llvm::LowerARMMachineInstrToMCInst?

Well spotted! I was coming to exactly the same conclusion myself a
couple of weekends back. Mihail was right that overhauling all of
CodeGen to use the new forms would be a very large task (I started it
on a branch, but hadn't/haven't finished yet).

> I’m sure I’m missing quite a lot of the details here, but would this
> approach seem reasonable to you?

Nope, it seems like a reasonable assessment. I'll see what I can do
this weekend.

Cheers.

Tim.




More information about the llvm-commits mailing list