[llvm] r215390 - [AArch64] Fix registerAllocator assigns same register for base and wback in

Quentin Colombet qcolombet at apple.com
Mon Aug 11 15:42:19 PDT 2014


Hi Chad,

On Aug 11, 2014, at 3:31 PM, Chad Rosier <mcrosier at codeaurora.org> wrote:

> Quentin,
> 
>> @@ -3017,7 +3017,6 @@ class StorePreIdx<bits<2> sz, bit V, bit
>> // Load/store post-indexed
>> //---
>> 
>> -// (pre-index) load/stores.
> 
> Was this change intentional?

Yes, it was. This template is for post-indexed load/store and the related comment is already two lines above.

I guess it could have been cleaned-up in a subsequent commit, though I missed that change during the review!!

Thanks for checking.

-Quentin
> 
> Chad
> 
> 
> 




More information about the llvm-commits mailing list