[PATCH] D130689: [LLVM] Update C++ standard to 17
Roy Jacobson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 7 10:13:38 PDT 2022
royjacobson added a comment.
This seems to have been more disruptive than expected, since an existing CMakeCache.txt can make LLVM compile in previous C++14 configuration. This seems to make some of the bots fail in a way that makes the patches making use of C++17 features seem at fault.
How would you feel about adding something like
#if defined(__cplusplus) && __cplusplus < 201703L
#error "LLVM requires at least C++17"
to some central header, to make this switch more visible?
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits