[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