[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