RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2025-02-01 09:53:54 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[LowerSwitch] Use ConstantRange::fromKnownBits(); NFC

Browse Source 
Using an unsigned range to stay NFC, but a signed range would really
be more useful here.

llvm-svn: 356831
This commit is contained in:
Nikita Popov 2019-03-23 12:48:54 +00:00
parent 63dafb2876
commit 24d342fbdb
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/Transforms/Utils/LowerSwitch.cpp
View File

@ -436,14 +436,6 @@ unsigned LowerSwitch::Clusterify(CaseVector& Cases, SwitchInst *SI) {
return NumSimpleCases;
}
static ConstantRange getConstantRangeFromKnownBits(const KnownBits &Known) {
APInt Lower = Known.One;
APInt Upper = ~Known.Zero + 1;
if (Upper == Lower)
return ConstantRange(Known.getBitWidth(), /*isFullSet=*/true);
return ConstantRange(Lower, Upper);
}
/// Replace the specified switch instruction with a sequence of chained if-then
/// insts in a balanced binary search.
void LowerSwitch::processSwitchInst(SwitchInst *SI,
@ -501,7 +493,9 @@ void LowerSwitch::processSwitchInst(SwitchInst *SI,
// switch, while LowerSwitch only needs to call LVI once per switch.
const DataLayout &DL = F->getParent()->getDataLayout();
KnownBits Known = computeKnownBits(Val, DL, /*Depth=*/0, AC, SI);
ConstantRange KnownBitsRange = getConstantRangeFromKnownBits(Known);
// TODO Shouldn't this create a signed range?
ConstantRange KnownBitsRange =
ConstantRange::fromKnownBits(Known, /*ForSigned=*/false);
const ConstantRange LVIRange = LVI->getConstantRange(Val, OrigBlock, SI);
ConstantRange ValRange = KnownBitsRange.intersectWith(LVIRange);
// We delegate removal of unreachable non-default cases to other passes. In