[libc-commits] [PATCH] D118871: [libc] Properly setting -std=c++17 for targets that requires C++17.

[Tue Ly via Phabricator via libc-commits]

lntue added a comment.

In D118871#3292655 <https://reviews.llvm.org/D118871#3292655>, @abrachet wrote:

> If you a need a c++17 compliant compiler to build libm, should we just set it for all of libc? Looks like flang <https://github.com/llvm/llvm-project/blob/main/flang/CMakeLists.txt#L6> already does this.



In D118871#3292655 <https://reviews.llvm.org/D118871#3292655>, @abrachet wrote:

> If you a need a c++17 compliant compiler to build libm, should we just set it for all of libc? Looks like flang <https://github.com/llvm/llvm-project/blob/main/flang/CMakeLists.txt#L6> already does this.

That sounds good to me if @sivachandra is fine with pushing the whole libc to C++17.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118871