[PATCH] D80047: Don't tail merge EHPads
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 17 00:29:25 PDT 2020
efriedma added reviewers: jyknight, efriedma.
efriedma added a comment.
I think it's specifically the tailIsWholeBlock() cases that require more care to handle your example correctly, in BranchFolder::TryTailMergeBlocks. The general transform is fine. -verify-machineinstrs should catch the transform in your example, I think.
Well, there are also potential issues on Windows involving funclets, but it looks like that isn't the problem here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80047/new/
https://reviews.llvm.org/D80047
More information about the llvm-commits
mailing list