[PATCH] D40948: Switch Clang's default C++ language target to C++14

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 7 11:44:56 PST 2017


probinson added a reviewer: rjmccall.
probinson added a comment.

+rjmccall for the codegen bits.



================
Comment at: clang/test/SemaCXX/new-array-size-conv.cpp:1
-// RUN: %clang_cc1 -fsyntax-only -pedantic -verify %s
+// RUN: %clang_cc1 -fsyntax-only -pedantic -verify -std=gnu++98 %s
 // RUN: %clang_cc1 -fsyntax-only -pedantic -verify -std=c++98 %s
----------------
I think the intent here was "whatever the current default is" and not specifically gnu++98, because the next line explicitly specifies c++98.  So, unless this test fails miserably for C++14 (which I wouldn't expect) I think it should be adapted to whatever gets reported for that dialect.


Repository:
  rC Clang

https://reviews.llvm.org/D40948





More information about the cfe-commits mailing list