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

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

Reply via email to