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

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 17 07:06:46 PST 2016


mcrosier added a comment.

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

> In http://reviews.llvm.org/D17097#350462, @t.p.northover wrote:
>
> > Can we have a test?
>
>
> The testcase added in http://reviews.llvm.org/D17098 contains two functions @st2 and @st4 that would not contain two stp without the current change.


I'd much prefer these type of tests over grepping debug output.  If @t.p.northover doesn't object, I'm fine with landing this patch as is and include the test cases in the followup patch.


http://reviews.llvm.org/D17097





More information about the llvm-commits mailing list