[PATCH] D80047: Don't tail merge EHPads
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat May 16 14:17:58 PDT 2020
efriedma added a comment.
Could you explain why you think this is the right thing to do?
At first glance, I don't see any obvious problem with tail merging in this context unless this is illegal due to "region" restrictions (merging code in different unwind handlers). And if the issue is in fact a problem with merging across regions, this fix is clearly insufficient; isEHPad() is only true for the first MBB of an unwind handler.
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