[PATCH] D42263: [ARM] Fix perf regression in compare optimization.
Joel Galenson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 18 13:14:02 PST 2018
jgalenson added a comment.
The problem was the "--E" in my previous commit (about 25 lines above the changed line). That caused the loop just below it (containing this line) to run in some cases it did not before. If I is the first instruction in the block, it now enters the loop and then returns false at the line I changed. I modified that to break out of the loop instead, which allows us to still consider MI as a candidate.
https://reviews.llvm.org/D42263
More information about the llvm-commits
mailing list