[libcxx-commits] [libcxx] [libc++] Remove a duplicated definition of _LIBCPP_NOINLINE (PR #79114)

via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jan 23 02:05:50 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: Dmitri Gribenko (gribozavr)

<details>
<summary>Changes</summary>

I failed to delete the old definition as a part of https://github.com/llvm/llvm-project/pull/73838.

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


1 Files Affected:

- (modified) libcxx/include/__config (-6) 


``````````diff
diff --git a/libcxx/include/__config b/libcxx/include/__config
index eab37e7576bdc3e..9557e8e8cf97f28 100644
--- a/libcxx/include/__config
+++ b/libcxx/include/__config
@@ -1235,12 +1235,6 @@ __sanitizer_verify_double_ended_contiguous_container(const void*, const void*, c
 #    define _LIBCPP_CONSTINIT
 #  endif
 
-#  if __has_attribute(__noinline__)
-#    define _LIBCPP_NOINLINE __attribute__((__noinline__))
-#  else
-#    define _LIBCPP_NOINLINE
-#  endif
-
 #  if defined(__CUDACC__) || defined(__CUDA_ARCH__) || defined(__CUDA_LIBDEVICE__)
 // The CUDA SDK contains an unfortunate definition for the __noinline__ macro,
 // which breaks the regular __attribute__((__noinline__)) syntax. Therefore,

``````````

</details>


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


More information about the libcxx-commits mailing list