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

Sebastian Pop via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 10:28:25 PST 2016


sebpop added a comment.

In http://reviews.llvm.org/D17097#350017, @az wrote:

> In http://reviews.llvm.org/D17097#349303, @mcrosier wrote:
>
> > Do you have any performance results?
>
>
> For the performance testing, our local benchmark shows that this patch improved performance for 7 tests. The improvement ranges between 0.5% and 3%. We also have few noises (very small improvement or degradation).


To be clear, we tested both http://reviews.llvm.org/D17097 and http://reviews.llvm.org/D17098 together.  I don't think just http://reviews.llvm.org/D17097 by itself would have this impact.
With http://reviews.llvm.org/D17098 we have seen several places where the MI scheduler was able to move insns past ldp/stp, as can be seen in the changes we had to do to the tests.


http://reviews.llvm.org/D17097





More information about the llvm-commits mailing list