[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