[PATCH] D156597: [Clang] Handle static_assert messages with an expression started by a literal

2023-08-01 Thread Corentin Jabot via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG4d494e76d098: [Clang] Handle static_assert messages with an expression started by a literal (authored by cor3ntin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D156597: [Clang] Handle static_assert messages with an expression started by a literal

2023-07-31 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin updated this revision to Diff 545627. cor3ntin added a comment. Typo + format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156597/new/ https://reviews.llvm.org/D156597 Files: clang/lib/Parse/ParseDeclCXX.cpp

[PATCH] D156597: [Clang] Handle static_assert messages with an expression started by a literal

2023-07-31 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from minor nits. Comment at: clang/lib/Parse/ParseDeclCXX.cpp:1022 + for(unsigned I = 0; ; ++I) { +const Token & T =

[PATCH] D156597: [Clang] Handle static_assert messages with an expression started by a literal

2023-07-31 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin added inline comments. Comment at: clang/lib/Parse/ParseDeclCXX.cpp:1019 -if (isTokenStringLiteral()) - AssertMessage = ParseUnevaluatedStringLiteralExpression(); -else if (getLangOpts().CPlusPlus26) +bool ParseAsExression = false; +

[PATCH] D156597: [Clang] Handle static_assert messages with an expression started by a literal

2023-07-29 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder added inline comments. Comment at: clang/lib/Parse/ParseDeclCXX.cpp:1019 -if (isTokenStringLiteral()) - AssertMessage = ParseUnevaluatedStringLiteralExpression(); -else if (getLangOpts().CPlusPlus26) +bool ParseAsExression = false; +

[PATCH] D156597: [Clang] Handle static_assert messages with an expression started by a literal

2023-07-29 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin created this revision. Herald added a project: All. cor3ntin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D156597 Files: clang/lib/Parse/ParseDeclCXX.cpp