[PATCH] D147867: [Windows SEH] Fix ehcleanup crash for Windows -EHa

[Eli Friedman via Phabricator via cfe-commits]

efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: clang/test/CodeGen/windows-seh-EHa-CppCatchReturn.cpp:27
+
+// FIXME: We may need to generate llvm.seh.scope.end or remove llvm.seh.scope.begin.
+void qux() {
----------------
If a scope never ends because there's an "unreachable", that's fine, I think.  The scope never dynamically ends (unless an exception is thrown).  The backend deals with the same sort of thing for other constructs with a "begin" and "end".  For example, a cleanuppad might not have a matching cleanupret.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147867/new/

https://reviews.llvm.org/D147867