[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