[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