[libcxx-commits] [libcxx] Add 64bit atomic check in the is_always_lock_free_pass test. (PR #111540)

via libcxx-commits libcxx-commits at lists.llvm.org
Tue Oct 8 07:56:21 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: None (simpal01)

<details>
<summary>Changes</summary>

Currently this test is completely xfailed as part of the patch https://github.com/llvm/llvm-project/pull/106077. But this test works on A and R profile, not in v7M profile. Because the test contain cases in which  m-profile will fail for atomic types greater than 4 bytes in size.

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


1 Files Affected:

- (modified) libcxx/test/std/atomics/atomics.lockfree/is_always_lock_free.pass.cpp (+1-1) 


``````````diff
diff --git a/libcxx/test/std/atomics/atomics.lockfree/is_always_lock_free.pass.cpp b/libcxx/test/std/atomics/atomics.lockfree/is_always_lock_free.pass.cpp
index 1ebe31375079b8..e922bc7413514a 100644
--- a/libcxx/test/std/atomics/atomics.lockfree/is_always_lock_free.pass.cpp
+++ b/libcxx/test/std/atomics/atomics.lockfree/is_always_lock_free.pass.cpp
@@ -7,7 +7,7 @@
 //===----------------------------------------------------------------------===//
 
 // UNSUPPORTED: c++03, c++11, c++14
-// XFAIL: LIBCXX-PICOLIBC-FIXME
+// XFAIL: !has-64-bit-atomics
 
 // <atomic>
 //

``````````

</details>


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


More information about the libcxx-commits mailing list