[libcxx-commits] [libcxx] [libc++][NFC] Use TestEachPointerType in TestEachAtomicType (PR #91480)
via libcxx-commits
libcxx-commits at lists.llvm.org
Wed May 8 07:41:51 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-libcxx
Author: Louis Dionne (ldionne)
<details>
<summary>Changes</summary>
That way, if we ever expand TestEachPointerType we will pick up those changes in TestEachAtomicType.
---
Full diff: https://github.com/llvm/llvm-project/pull/91480.diff
1 Files Affected:
- (modified) libcxx/test/support/atomic_helpers.h (+1-2)
``````````diff
diff --git a/libcxx/test/support/atomic_helpers.h b/libcxx/test/support/atomic_helpers.h
index 9a32b1ffe85e4..0266a0961067b 100644
--- a/libcxx/test/support/atomic_helpers.h
+++ b/libcxx/test/support/atomic_helpers.h
@@ -116,6 +116,7 @@ template <template <class TestArg> class TestFunctor>
struct TestEachAtomicType {
void operator()() const {
TestEachIntegralType<TestFunctor>()();
+ TestEachPointerType<TestFunctor>()();
TestFunctor<UserAtomicType>()();
/*
Note: These aren't going to be lock-free,
@@ -128,8 +129,6 @@ struct TestEachAtomicType {
TestFunctor<PaddedUserAtomicType>()();
TestFunctor<WeirdUserAtomicType>()();
*/
- TestFunctor<int*>()();
- TestFunctor<const int*>()();
TestFunctor<float>()();
TestFunctor<double>()();
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/91480
More information about the libcxx-commits
mailing list