[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