[X86] Remove ANY_EXTEND special case from canReduceVMulWidth
authorCraig Topper <craig.topper@intel.com>
Thu, 15 Nov 2018 21:19:32 +0000 (21:19 +0000)
committerCraig Topper <craig.topper@intel.com>
Thu, 15 Nov 2018 21:19:32 +0000 (21:19 +0000)
commit22bfa99448ec2d706ddb6a831e1b1f0d2ea5c524
tree7d45d70d84d6d22187b0eb273b2cce3352129994
parent04f9d0a0c31398651ecf52a853362aa328756011
[X86] Remove ANY_EXTEND special case from canReduceVMulWidth

Removing this code doesn't affect any lit tests so it doesn't appear to be tested anymore. I assume it was when it was added, but I guess something else changed? Code coverage report also says its unused.

I mostly didn't like that it seemed to count the sign bits as if it was a sign_extend, but then set isPositive as if it was a zero_extend. It feels like we should have picked one interpretation?

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

llvm-svn: 346995
llvm/lib/Target/X86/X86ISelLowering.cpp