[PATCH] D133500: [clang] Correct handling of lambdas in lambda default arguments in dependent contexts.

Tom Honermann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 1 05:13:51 PDT 2022


tahonermann marked 5 inline comments as done.
tahonermann added inline comments.


================
Comment at: clang/lib/Sema/SemaTemplateInstantiate.cpp:1159
+        Expr *UninstExpr = PVD->getUninstantiatedDefaultArg();
+        // FIXME: Obtain the source location for the '=' token.
+        SourceLocation EqualLoc = UninstExpr->getBeginLoc();
----------------
erichkeane wrote:
> Looks like we don't store the equals-token location, so any attempt here probably would have to have ParmVarDecl store this location somewhere?
That's right. This is a pre-existing issue; there are several places that have this same (or similar) FIXME comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133500



More information about the cfe-commits mailing list