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

Tobias Hieta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 7 12:55:33 PDT 2022


thieta added a comment.

In D130689#3705236 <https://reviews.llvm.org/D130689#3705236>, @royjacobson wrote:

> This affects people on their work branches as well, and it's not obvious that it's a configuration error and not a broken master.
>
> The CMake approach sounds cleaner to me, but I don't know CMake well enough to do it - if you could post a follow up patch I think it would be quite helpful.

Good point - I tried a few things and came up with an approach that works here: https://reviews.llvm.org/D131367 - have a look.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130689



More information about the cfe-commits mailing list