[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