[Openmp-commits] [PATCH] D74258: [OpenMP] Switch default C++ standard to C++ 14

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 marked an inline comment as done.
tianshilei1992 added inline comments.


================
Comment at: openmp/cmake/HandleOpenMPOptions.cmake:32-34
+append_if(OPENMP_HAVE_STD_GNUPP11_FLAG "-std=gnu++14" CMAKE_CXX_FLAGS)
 if (NOT OPENMP_HAVE_STD_GNUPP11_FLAG)
+  append_if(OPENMP_HAVE_STD_CPP11_FLAG "-std=c++14" CMAKE_CXX_FLAGS)
----------------
Hahnfeld wrote:
> You should rename this, otherwise an incremental build won't test the new flag IIRC.
Oh, exactly, sorry for missing that.


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

https://reviews.llvm.org/D74258