[PATCH] D66067: Push lambda scope earlier when transforming lambda expression

Mark de Wever via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 17 11:35:41 PDT 2019


Mordante added inline comments.


================
Comment at: clang/test/SemaTemplate/default-arguments-cxx0x.cpp:1
-// RUN: %clang_cc1 -fsyntax-only -std=c++11 -verify %s
+// RUN: %clang_cc1 -fsyntax-only -std=c++14 -verify %s
 // expected-no-diagnostics
----------------
Wouldn't it be better to keep the original C++11 test and add a second `RUN:` for the C++14 test?
Then guard the new test with `#if __cplusplus >= 201402L`


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66067/new/

https://reviews.llvm.org/D66067





More information about the cfe-commits mailing list