[PATCH] D77982: [Windows SEH] Fix the frame-ptr of a nested-filter within a _finally

[Eli Friedman via Phabricator via cfe-commits]

efriedma added a comment.

For (1), I can see your point that it's sort of a balancing act.  But generally, I'm concerned about making fragile assumptions: here, that LocalDeclMap contains precisely the two ImplicitParmDecls for the arguments, and nothing else.  If we are going to assume that, I'd prefer stronger assertions so we don't accidentally break that assumption in the future.

re: (2), I don't have a problem using the LocalDeclMap, just the part where you scan it using some fragile assumptions.  I was thinking you would save the ImplicitParmDecl*, not the actual alloca.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77982