]> granicus.if.org Git - llvm/commit
[InstCombine] Remove redundant combine from visitAnd
authorCraig Topper <craig.topper@gmail.com>
Sun, 2 Apr 2017 17:34:30 +0000 (17:34 +0000)
committerCraig Topper <craig.topper@gmail.com>
Sun, 2 Apr 2017 17:34:30 +0000 (17:34 +0000)
commit9e3a09349cb94f0f623863927bd566b378411313
tree7551ac149f04526d5da0f1d54c8ea0b42aa6a8cf
parent888b18156656072faa8ef151a234aed383f9d410
[InstCombine] Remove redundant combine from visitAnd

As far as I can tell this combine is fully handled by SimplifyDemandedInstructionBits.

I was only looking at this because it is the only user of APIntOps::isShiftedMask which is itself broken. As demonstrated by r299187. I was going to fix isShiftedMask and needed to make sure we had coverage for the new cases it would expose to this combine. But looks like we can nuke it instead.

Differential Revision: https://reviews.llvm.org/D31543

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@299337 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
lib/Transforms/InstCombine/InstCombineInternal.h