[PATCH] D113497: [IPSCCP] Support unfeasible default dests for switch.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 29 03:09:05 PST 2021


fhahn added a comment.

In D113497#3155755 <https://reviews.llvm.org/D113497#3155755>, @lebedev.ri wrote:

> LG.
>
> If SCCP runs again, will it then again create a new unreachable block
> for the default that is already pointing at the unreachable block?
> Should it not do such redundant work?

Good point. In practice I think IPSCCP is only run twice when doing LTO, so it's probably not a big deal. But let me check if the default dst is already unreachable.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113497/new/

https://reviews.llvm.org/D113497



More information about the llvm-commits mailing list