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

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

> Unless there are good reasons to switch, I'd object because it requires newer versions of compilers than many HPC systems have.

I disagree. The use case we would be supporting is:

- building llvm libomptarget from source
- using it with a binary llvm dist, or some other toolchain
- won't download or build a newer toolchain

The advantage isn't in make_unique, it's in matching the rest of llvm which has required c++14 for months. It's not worth rehashing the stability-vs-progress discussion here when the parent project has already reached consensus.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74258