[libcxx-commits] [libcxx] [libcxx] Do not warn on missing `ELAST` definition on LLVM libc (PR #109205)

via libcxx-commits libcxx-commits at lists.llvm.org
Wed Sep 18 14:54:02 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: Joseph Huber (jhuber6)

<details>
<summary>Changes</summary>

Summary:
The LLVM C library shouldn't need this `ELAST` definition because our
implementation should handle out-of-range error values gracefully.


---
Full diff: https://github.com/llvm/llvm-project/pull/109205.diff


1 Files Affected:

- (modified) libcxx/src/include/config_elast.h (+2) 


``````````diff
diff --git a/libcxx/src/include/config_elast.h b/libcxx/src/include/config_elast.h
index 899e124ad261b2..28fc9ca938a377 100644
--- a/libcxx/src/include/config_elast.h
+++ b/libcxx/src/include/config_elast.h
@@ -31,6 +31,8 @@
 // No _LIBCPP_ELAST needed on WASI
 #elif defined(__EMSCRIPTEN__)
 // No _LIBCPP_ELAST needed on Emscripten
+#elif defined(__LLVM_LIBC__)
+// No _LIBCPP_ELAST needed on LLVM's libc.
 #elif defined(__linux__) || defined(_LIBCPP_HAS_MUSL_LIBC)
 #  define _LIBCPP_ELAST 4095
 #elif defined(__APPLE__)

``````````

</details>


https://github.com/llvm/llvm-project/pull/109205


More information about the libcxx-commits mailing list