v.g.vassilev added inline comments.
================
Comment at: clang/lib/Parse/ParseTentative.cpp:94
+ // involve namespaces and friend declarations.
+ if (NextToken().is(tok::identifier))
+ return true;
----------------
I am wondering what is the false positive rate of this change? That is, if we
enable incremental parsing by default (in a local build) and then run all tests
which do not produce diagnostics.
================
Comment at: clang/test/Interpreter/disambiguate-decl-stmt.cpp:74
+// expected-error@-1 2 {{'Inner' is a private member of 'PR13642'}}
+
// Deduction guide
----------------
Could we move the diagnostic-producing cases in a separate file?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157838/new/
https://reviews.llvm.org/D157838
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
