[PATCH] D52064: [Sema] Add a note pointing to the first use of an implicit capture

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added inline comments.


================
Comment at: clang/lib/Sema/SemaLambda.cpp:1427
   if (RefResult.isInvalid())
     return ExprError();
   Expr *Ref = RefResult.get();
----------------
This early exit leaves your CodeSynthesisContext on the stack. Consider using 
RAII?


================
Comment at: clang/test/SemaCXX/lambda-expressions.cpp:87
+    (void)^{ // expected-error@+1 {{no matching constructor for initialization 
of 'const G'}}
+      return [=]{ // expected-error@+1 {{no matching constructor for 
initialization of 'const G'}}
+        const G* gg = &g; // expected-note {{implicitly capturing 'g', first 
used here}}
----------------
Why are these @+1?


https://reviews.llvm.org/D52064



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits