[PATCH] D124434: [Clang][Test] Run tests in C++14 mode explicitly.

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 26 09:49:26 PDT 2022

dblaikie added subscribers: aaron.ballman, rsmith.
dblaikie added a comment.

@rsmith @aaron.ballman - might be especially interesting to know your thoughts on the C++ chapter-based testing and what the intent there is as clang changes default versions/new versions are added. (& also whether there's an approach (like moving the default language into the driver) that'd be worthwhile to lower the maintenance cost of these sort of migrations without compromising on test coverage/ending up with all the test coverage testing previous versions of the language and not what everyone's using in the wild)

Comment at: clang/test/CXX/class.access/class.friend/p1.cpp:4
 // RUN: %clang_cc1 -fsyntax-only -verify -std=c++14 %s
-// RUN: %clang_cc1 -fsyntax-only -verify %s
Might be worth knowing the version history of tests like this? (in here and other test files) - perhaps the intent is for this test to check the code is valid in all language versions & so should grow more explicit version lines as the language evolves?

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list