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

Renato Golin renato.golin at linaro.org
Thu Jun 20 16:07:44 PDT 2013


Thanks Quentin!

Renato
On 20 Jun 2013 23:55, "Quentin Colombet" <qcolombet at apple.com> wrote:

> 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/20130621/9f7bcd25/attachment.html>


More information about the llvm-commits mailing list