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

Andrew Trick atrick at apple.com
Thu Jun 20 15:36:43 PDT 2013


On Jun 20, 2013, at 2:49 PM, Renato Golin <renato.golin at linaro.org> wrote:

> On 20 June 2013 17:59, Quentin Colombet <qcolombet at apple.com> wrote:
> Ping?
> 
> Hi Quentin,
> 
> This is dragging for a bit too long... Andrew seemed to have a good idea of what that was about, and he was fine with the change.
> 
> I'm happy with the patch if he's happy. Evan can always to a post-commit check later if needed, and we have buildbots to help us with the testing.
> 
> If no one else objects, I think you can commit, keep an eye and see where it leads us.

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.

-Andy

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


More information about the llvm-commits mailing list