[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