[PATCH] D93943: [SimplifyCFG] Update SimplifyBranchOnICmpChain to recognize select form of and/or
Juneyoung Lee via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 31 12:01:26 PST 2020
aqjune added a comment.
In D93943#2475598 <https://reviews.llvm.org/D93943#2475598>, @nikic wrote:
> I think this may not be correct if there is an `ExtraCase`. `ExtraCase || X == C || ...` should be fine as it is converted into `if (!ExtraCase) goto; switch (X)`, but something like `... || X == C || ExtraCase` would not be fine.
You're right. Actually, while updating test/Transforms/SimplifyCFG/switch_create.ll I realized this and maybe I wanted to think that it wasn't very important, *but* that's not the right direction. I'll update this patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93943/new/
https://reviews.llvm.org/D93943
More information about the llvm-commits
mailing list