[InstCombine] Speculatively implement a fix for what might be the root cause of PR337...
authorCraig Topper <craig.topper@intel.com>
Sun, 9 Jul 2017 03:25:17 +0000 (03:25 +0000)
committerCraig Topper <craig.topper@intel.com>
Sun, 9 Jul 2017 03:25:17 +0000 (03:25 +0000)
commite79b3e7d9a26808b3e0b58a73076cf2bb9a1c9cc
treeef2b047a52c9460baf1758ee508c751cda9cc9d3
parentd362d27c27697e258816d01b962476b51663562f
[InstCombine] Speculatively implement a fix for what might be the root cause of PR33721 by making sure that we have integer types before doing select C, -1, 0 -> sext C to int

I recently changed m_One and m_AllOnes to use Constant::isOneValue/isAllOnesValue which work on floating point values too. The original implementation looked specifically for ConstantInt scalars and splats. So I'm guessing we are accidentally trying to issue sext/zexts on floating point types now.

Hopefully I figure out how to reproduce the failure from the PR soon.

llvm-svn: 307486
llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp