[PATCH] D119854: [SimplifyCFG][PhaseOrdering] Defer lowering switch into an integer range comparison and branch until after at least the IPSCCP
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 16 03:45:43 PST 2022
lebedev.ri added a comment.
In D119854#3325590 <https://reviews.llvm.org/D119854#3325590>, @nikic wrote:
> This looks sensible to me.
>
> Would it be sufficient to only block this transform in the very first (pre-inlining) SimplifyCFG run, which I think would allow IPSCCP to fold it? Or does only CVP handle this case?
For the tests //in question// it would appear IPSCCP is enough, so i've changed it accordingly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119854/new/
https://reviews.llvm.org/D119854
More information about the llvm-commits
mailing list