[PATCH] D158808: [Clang] Modify Parser::ParseLambdaExpressionAfterIntroducer to check whether the lambda-declarator is valid
Corentin Jabot via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 24 23:17:53 PDT 2023
cor3ntin added a comment.
LGTM modulo:
- No release notes
- I think you need to write in the commit
Fixes https://github.com/llvm/llvm-project/issues/64962
Fixes https://github.com/llvm/llvm-project/issues/28679
for github not to be confused
- You need to clang-format
================
Comment at: clang/test/Parser/cxx0x-lambda-expressions.cpp:2
+// RUN: %clang_cc1 -fsyntax-only -Wno-unused-value -verify -Wno-c++20-extensions -std=c++11 -Wno-c99-designator %s
// RUN: %clang_cc1 -fsyntax-only -Wno-unused-value -verify -std=c++20 -Wno-c99-designator %s
// RUN: %clang_cc1 -fsyntax-only -Wno-unused-value -verify -std=c++23 -Wno-c99-designator %s
----------------
We might want to put that test in `clang/test/Parser/cxx2a-template-lambdas.cpp` instead
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158808/new/
https://reviews.llvm.org/D158808
More information about the cfe-commits
mailing list