[llvm] [Support] Remove ciso646 include (PR #123578)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 20 02:13:14 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-support
Author: Nikita Popov (nikic)
<details>
<summary>Changes</summary>
This header has been removed in C++20 and causes a large amount of deprecation spam when building against libstdc++ 15 in C++17 mode.
As far as I understand, we just need to include *some* STL header to get access to the version macros, and as this header also includes <optional> nowadays we can just drop the <cstd646> include entirely.
---
Full diff: https://github.com/llvm/llvm-project/pull/123578.diff
1 Files Affected:
- (modified) llvm/include/llvm/Support/Threading.h (-1)
``````````diff
diff --git a/llvm/include/llvm/Support/Threading.h b/llvm/include/llvm/Support/Threading.h
index d8e2cb0514ddd7..01e26ad9b858ea 100644
--- a/llvm/include/llvm/Support/Threading.h
+++ b/llvm/include/llvm/Support/Threading.h
@@ -18,7 +18,6 @@
#include "llvm/ADT/StringRef.h"
#include "llvm/Config/llvm-config.h" // for LLVM_ON_UNIX
#include "llvm/Support/Compiler.h"
-#include <ciso646> // So we can check the C++ standard lib macros.
#include <optional>
#if defined(_MSC_VER)
``````````
</details>
https://github.com/llvm/llvm-project/pull/123578
More information about the llvm-commits
mailing list