[PATCH] D22360: [AArch64] Set COPY ZR isAsCheapAsAMove when needed.

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 12:00:04 PDT 2016


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.



> Unfortunately, several benchmarks have double-digit regression. However, on Kryo, setting any COPY isAsCheapAsAMove brings small win, but restricting it to COPY ZR is slightly better.


Wow! I would never expect that to have such a huge effect, we may have a bug or bad strategy somewhere in the generic codegen. It would certainly be interesting to learn why we have those huge swings...

In any case this patch LGTM.


Repository:
  rL LLVM

https://reviews.llvm.org/D22360





More information about the llvm-commits mailing list