[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