[PATCH] D68715: [mangle] Fix mangling where an extra mangle context is required.

Michael Liao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 9 11:54:55 PDT 2019


hliao marked an inline comment as done.
hliao added inline comments.


================
Comment at: clang/lib/Sema/SemaLambda.cpp:353
+                                          ManglingContextDecl),
+        ManglingContextDecl);
   }
----------------
efriedma wrote:
> If I'm following correctly, the core of the change is right here?  Instead of grabbing a context from the ExpressionEvaluationContextRecord directly, we instead grab the current decl from it, and look up the context from that?  That makes sense.
yeah, that's the code fix. Lots of changes are due to removing unused code due to this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68715





More information about the cfe-commits mailing list