[PATCH] D31543: [InstCombine] Remove redundant combine from visitAnd
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 4 09:00:10 PDT 2017
craig.topper reopened this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
I had to revert the commit of this due to a tsan check failure. it looks like this optimization does fire sometimes because it's mising a qualification to make sure the add or sub is only has one user. So we can create an add or sub without replacing the existing one.
Repository:
rL LLVM
https://reviews.llvm.org/D31543
More information about the llvm-commits
mailing list