[PATCH] D57581: Explicitly add language standard option to test cases that rely on the C++14 default

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 1 08:56:17 PST 2019


nemanjai added a comment.

In D57581#1380609 <https://reviews.llvm.org/D57581#1380609>, @ilya-biryukov wrote:

> ...
>  Do have a comment, though. Any, reason to use `-std=gnu++14` and not `-std=c++14`?
>  Most (all?) of the tests do not seem to have anything to do with the gnu extensions, so why enable them?


I only used `gnu++14` in the test cases to match the current default in clang (i.e. not change how clang handles the test case). But I have no problem whatsoever with making these standard C++.


Repository:
  rC Clang

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

https://reviews.llvm.org/D57581





More information about the cfe-commits mailing list