diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -254,6 +254,9 @@ requires on lambdas when not allowed, which we previously missed. (`#61748 `_) - Fix confusing diagnostic for incorrect use of qualified concepts names. +- Fix handling of comments in function like macros so they are ignored in -CC + mode. + (`#60887 `_) Bug Fixes to Compiler Builtins diff --git a/clang/lib/Lex/PPDirectives.cpp b/clang/lib/Lex/PPDirectives.cpp --- a/clang/lib/Lex/PPDirectives.cpp +++ b/clang/lib/Lex/PPDirectives.cpp @@ -2643,7 +2643,7 @@ SmallVector Parameters; while (true) { - LexUnexpandedToken(Tok); + LexUnexpandedNonComment(Tok); switch (Tok.getKind()) { case tok::r_paren: // Found the end of the parameter list. @@ -2664,7 +2664,7 @@ } // Lex the token after the identifier. - LexUnexpandedToken(Tok); + LexUnexpandedNonComment(Tok); if (Tok.isNot(tok::r_paren)) { Diag(Tok, diag::err_pp_missing_rparen_in_macro_def); return true; @@ -2698,7 +2698,7 @@ Parameters.push_back(II); // Lex the token after the identifier. - LexUnexpandedToken(Tok); + LexUnexpandedNonComment(Tok); switch (Tok.getKind()) { default: // #define X(A B @@ -2714,7 +2714,7 @@ Diag(Tok, diag::ext_named_variadic_macro); // Lex the token after the identifier. - LexUnexpandedToken(Tok); + LexUnexpandedNonComment(Tok); if (Tok.isNot(tok::r_paren)) { Diag(Tok, diag::err_pp_missing_rparen_in_macro_def); return true; diff --git a/clang/test/Preprocessor/comment_save_macro.c b/clang/test/Preprocessor/comment_save_macro.c --- a/clang/test/Preprocessor/comment_save_macro.c +++ b/clang/test/Preprocessor/comment_save_macro.c @@ -1,13 +1,28 @@ // RUN: %clang_cc1 -E -C %s | FileCheck -check-prefix=CHECK-C -strict-whitespace %s // CHECK-C: boo bork bar // zot +// CHECK-C: ( 0 ); +// CHECK-C: ( 0,1,2 ); +// CHECK-C: ( 0,1,2 ); // RUN: %clang_cc1 -E -CC %s | FileCheck -check-prefix=CHECK-CC -strict-whitespace %s // CHECK-CC: boo bork /* blah*/ bar // zot +// CHECK-CC: (/**/0/**/); +// CHECK-CC: (/**/0,1,2/**/); +// CHECK-CC: (/**/0,1,2/**/); // RUN: %clang_cc1 -E %s | FileCheck -strict-whitespace %s // CHECK: boo bork bar +// CHECK: ( 0 ); +// CHECK: ( 0,1,2 ); +// CHECK: ( 0,1,2 ); #define FOO bork // blah boo FOO bar // zot +#define M(/**/x/**/) (/**/x/**/) +M(0); +#define M2(/**/.../**/) (/**/__VA_ARGS__/**/) +M2(0,1,2); +#define M3(/**/x.../**/) (/**/x/**/) +M3(0,1,2);