diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-07-09 03:25:17 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-07-09 03:25:17 +0000 |
| commit | e79b3e7d9a26808b3e0b58a73076cf2bb9a1c9cc (patch) | |
| tree | ef2b047a52c9460baf1758ee508c751cda9cc9d3 /llvm/lib/Transforms/IPO/Inliner.cpp | |
| parent | d362d27c27697e258816d01b962476b51663562f (diff) | |
| download | bcm5719-llvm-e79b3e7d9a26808b3e0b58a73076cf2bb9a1c9cc.tar.gz bcm5719-llvm-e79b3e7d9a26808b3e0b58a73076cf2bb9a1c9cc.zip | |
[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
Diffstat (limited to 'llvm/lib/Transforms/IPO/Inliner.cpp')
0 files changed, 0 insertions, 0 deletions
