[PATCH] D17097: [AArch64] add mem ref to unscaled pairs

[Chad Rosier via llvm-commits]

mcrosier added a comment.

In http://reviews.llvm.org/D17097#349477, @sebpop wrote:

> Actually this change does add new ldp/stp pairs as it enables the mayAlias analysis to reason about the memref.
>  Of course you need the next patch http://revioews.llvm.org/D17098 to make the dependence analysis functional.
>  Thus the testcase for this change comes with that next patch.


Ah, correct.  With http://reviews.llvm.org/D17098 it will create more paired instructions.

> For the PR you mention, do you mean https://llvm.org/bugs/show_bug.cgi?id=26358

>  ?


Yes.  From the PR "Then mergePairedInsns() in the AArch64 load store optimizer needs to be fixed so that the pair instructions include their MMOs."


http://reviews.llvm.org/D17097