[PATCH] D53190: ARM: avoid infinite combining loop

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 27 02:59:27 PST 2018


samparker added a comment.

My point was that, even though this extra work and as you mentioned in the comments, the test case isn't generating optimum code. We could introduce a subs node to remove the unnecessary cmp, making the sub opaque and improving codegen. Unless there's a reason why we couldn't do this?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D53190/new/

https://reviews.llvm.org/D53190





More information about the llvm-commits mailing list