hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.

Thanks!



================
Comment at: clang/test/Parser/cxx-decltype-parameter.cpp:1
+// RUN: %clang_cc1 -fsyntax-only -verify %s
+
----------------
I'd place this test in `Parser/cxx-keyword-identifiers.cpp` since the crash is 
in the relevant code path.


================
Comment at: clang/test/Parser/cxx-decltype-parameter.cpp:5
+struct Foo {
+  Bar bar(*decltype(Bar{}) aux); // expected-error {{C++ requires a type 
specifier for all declarations}}. \
+                                 // expected-error {{expected ')'}} 
expected-note {{to match this '('}}
----------------
this can be simplified further:  `void bar(*decltype(1) aux);`




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114609

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

Reply via email to