[PATCH] D119266: [SimplifyCFG] Do not merge landingpads for invokes with deopt state.

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 8 11:13:32 PST 2022


dantrushin added a comment.

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

> I'm not sure i understand why this is required. Going through all the invokes
> and ensuring that every `deopt` `invoke` has a dedicated (empty) `landingpad`
> seems both trivial to do and something that you'd want to do a part of lowering later on anyway?.

I want to avoid doing optimization which I will undo later.
In our downstream pipeline repeated cycles of simplifycfg + loopsimplify (which undoes this transform, too) finally triggers opt blowup (which is different problem, yes).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119266



More information about the llvm-commits mailing list