]> granicus.if.org Git - llvm/commit
[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)
commit1e0e5aa155d9439a0c9d741d723a0339ea1634cd
tree01fbd694a12438855f4e24860793ea08fa7e43d8
parentdb24b6e4f77160645339598912f49397a6616870
[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.

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