Skip to content

Commit 7c8dcd0

Browse files
committed
Address feedback
1 parent 2288724 commit 7c8dcd0

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

llvm/lib/Transforms/Utils/SimplifyCFG.cpp

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5868,17 +5868,14 @@ bool SimplifyCFGOpt::turnSwitchRangeIntoICmp(SwitchInst *SI,
58685868
/*Cases=*/&CasesB,
58695869
/*OtherCases=*/&CasesA,
58705870
};
5871-
58725871
// Correctness: Cases to the default destination cannot be contiguous cases.
5873-
if (!ContiguousCases && !HasDefault && !CasesA.empty())
5874-
if (auto Result = findContiguousCases(SI->getCondition(), CasesA, CasesB,
5875-
DestA, DestB))
5876-
ContiguousCases = *Result;
5872+
else if (!HasDefault)
5873+
ContiguousCases =
5874+
findContiguousCases(SI->getCondition(), CasesA, CasesB, DestA, DestB);
58775875

58785876
if (!ContiguousCases)
5879-
if (auto Result = findContiguousCases(SI->getCondition(), CasesB, CasesA,
5880-
DestB, DestA))
5881-
ContiguousCases = *Result;
5877+
ContiguousCases =
5878+
findContiguousCases(SI->getCondition(), CasesB, CasesA, DestB, DestA);
58825879

58835880
if (!ContiguousCases)
58845881
return false;

0 commit comments

Comments
 (0)