[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