[llvm] [SimplifyCFG] Find the minimal table considering overflow in `switchToLookupTable` (PR #67885)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Sep 30 23:29:34 PDT 2023
================
@@ -6519,17 +6518,60 @@ static bool SwitchToLookupTable(SwitchInst *SI, IRBuilder<> &Builder,
SmallDenseMap<PHINode *, Type *> ResultTypes;
SmallVector<PHINode *, 4> PHIs;
- for (SwitchInst::CaseIt E = SI->case_end(); CI != E; ++CI) {
- ConstantInt *CaseVal = CI->getCaseValue();
- if (CaseVal->getValue().slt(MinCaseVal->getValue()))
- MinCaseVal = CaseVal;
- if (CaseVal->getValue().sgt(MaxCaseVal->getValue()))
- MaxCaseVal = CaseVal;
+ SmallVector<ConstantInt *, 8> CaseVals;
+ for (auto CI : SI->cases()) {
+ ConstantInt *CaseVal = CI.getCaseValue();
+ CaseVals.push_back(CaseVal);
+ }
+
+ // We want to find a range of indexes that will create the minimal table.
+ // We can treat all possible index values as a circle. For example, the i8 is
+ // [-128, -1] and [0, 127]. After that find the minimal range from this circle
----------------
DianQK wrote:
Updated.
> When considering overflow, the next indexed value of the maximum signed integer is the minimum signed integer.
https://github.com/llvm/llvm-project/pull/67885
More information about the llvm-commits
mailing list