[PATCH] Fix a bug in the Thumb1 ARM Load/Store optimizer which resulted in the base register reset instruction being incorrectly placed.
Moritz Roth
moritz.roth at arm.com
Tue Jun 10 08:53:32 PDT 2014
>>! In D4084#7, @rengolin wrote:
> Hi Morizt,
>
> Apart from the comment, LGTM.
>
> --renato
Hi Renato,
this was also a bug (which resulted in this issue happening in a few extra cases as far as I can tell) - we do
Opcode = getLoadStoreMultipleOpcode(Opcode, Mode);
a few lines before this check so Opcode can never be tLDRi (it would have been changed to tLDMIA).
I don't have commit access by the way, so someone else will have to commit this on my behalf.
Cheers
Moritz
http://reviews.llvm.org/D4084
More information about the llvm-commits
mailing list