arphaman added inline comments.

================
Comment at: lib/Parse/ParseExprCXX.cpp:1020
 
-  PA.Commit();
-  return false;
+  PA2.Revert();
+  return true;
----------------
Just to clarify: It seems like this revert (in addition to two TPAs) is the 
main addition. Did we hit the assertion previously with the crash?


================
Comment at: test/Parser/objcxx11-invalid-lambda.cpp:11
+}               // expected-error{{expected ')'}} expected-error{{expected ';' 
at end of declaration}} expected-error{{expected '}'}}
\ No newline at end of file

----------------
Nit: missing newline.


https://reviews.llvm.org/D39419



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

Reply via email to