[PATCH] D130689: [LLVM] Update C++ standard to 17

Mon Aug 8 14:45:05 PDT 2022

mstorsjo added a comment.

In D130689#3700094 <https://reviews.llvm.org/D130689#3700094>, @thakis wrote:

> In D130689#3696186 <https://reviews.llvm.org/D130689#3696186>, @thieta wrote:
>
>> @nikic @thakis I fixed this issue in https://reviews.llvm.org/D131063 and it can be built with CXX_STANDARD=17 and OSX_DEPLOYMENT_TARGET=10.11.
>
> Thanks! We took this opportunity to bump our clang deployment target from 10.7 to 10.12 (which makes a few other minor things easier too), so we don't need this change any more. But it's a good change anyways :)

FWIW I ran into an issue when doing Universal macOS builds (by building for arm64 and x86_64 at the same time) where I now had to raise my deployment target to 10.14 - see https://github.com/llvm/llvm-project/issues/57017 for more details about that.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130689