[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