Skip to content

Commit 2d61cc2

Browse files
michael-kenzelIcohedron
authored andcommitted
Remove dependence on <ciso646> (llvm#73273)
C++23 removed `<ciso646>` from the standard library. The header is used in two places: Once in order to pull in standard library macros. Since this file also includes `<optional>`, that use of `<ciso646>` is technically redundant, but should probably be left in in case a future change ever removes the include of `<optional>`. A second use of `<ciso646>` appears to have been introduced in da65009, but seems unnecessary (the file doesn't seem to use anything from that header, and it seems to build just fine on MSVC here without it). The new `<version>` header should be supported by all supported implementations. This change replaces uses of `<ciso646>` with the `<version>` header, or removes them entirely where unnecessary.
1 parent 6f7a24f commit 2d61cc2

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

llvm/include/llvm/Support/Threading.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
#include "llvm/Config/llvm-config.h" // for LLVM_ON_UNIX
2020
#include "llvm/Support/Compiler.h"
2121
#include <optional>
22+
#include <version>
2223

2324
#if defined(_MSC_VER)
2425
// MSVC's call_once implementation worked since VS 2015, which is the minimum

0 commit comments

Comments
 (0)