[PATCH] ARM: Form paired load after stack coloring.

Quentin Colombet qcolombet at apple.com
Thu Jun 20 15:55:26 PDT 2013


Thanks again for your reviews.

Committed as r184492.

I will send another mail with the PR number for the register allocation limitation exposed with this patch (as soon as I have filed it).

Cheers,

-Quentin

On Jun 20, 2013, at 3:44 PM, Quentin Colombet <qcolombet at apple.com> wrote:

> 
> On Jun 20, 2013, at 3:42 PM, Renato Golin <renato.golin at linaro.org> wrote:
> 
>> On 20 June 2013 23:36, Andrew Trick <atrick at apple.com> wrote:
>> I couldn’t see a problem with Quentin’s patch. Although we may want to add a comment to the effect that ignoring the memoperand’s value means we can’t rely on AliasAnalysis.
>> 
>> Would that break/limit any current AA? Or just future possible implementations?
>> 
>> I'm fine with the latter, but not so much with the former...
> From my understanding, the latter as we do not use AA here.
> 
> -Quentin
> 
>> 
>> cheers,
>> --renato
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130620/023e8117/attachment.html>


More information about the llvm-commits mailing list