[llvm] [SimplifyCFG] Don't limit the number of simultaneous forwards from switch condition (PR #95932)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 01:23:03 PDT 2024


DianQK wrote:

> (I can't access the alive2 links, the server seems to be down?)

Fixed. GitHub seems to not recognize `_`.

> I do think my diff above would fix your use case? So if we do decide to "always forward" I think we need some motivation, and we should update the comment for `FindPHIForConditionForwarding` with that motivation rather than talk about folding away destination blocks.

I think so, but it would also make this part of the code a bit more complex. In fact, I don't have a clear choice for the change here.

https://github.com/llvm/llvm-project/pull/95932


More information about the llvm-commits mailing list