[PATCH] D21299: [Codegen Prepare] Swap commutative binops before splitting branch condition.
Balaram Makam via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 11 05:09:15 PDT 2016
bmakam added a comment.
It seems like there is no objection to turn this on for A57 but there is no official approval yet. If it helps to pursuade the community, I have completed running tests on A53 and this change is performance neutral on A53 with no noise regressions or gains. IMHO this seems to be good for AArch64 targets, but I am inclined to leave it enabled only for Kryo because there is no approval from the community. Any thoughts?
http://reviews.llvm.org/D21299
More information about the llvm-commits
mailing list