[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