[libcxx-commits] [libcxx] [libc++][test] disable all atomic<long double> tests (PR #74201)
via libcxx-commits
libcxx-commits at lists.llvm.org
Sun Dec 3 07:29:50 PST 2023
huixie90 wrote:
> This is not an OK way to deal with broken tests.
There are lots of discussions on the other PR when these tests were added. Basically long double atomic is broken for years and we had some bug reports (against clang) open already. These tests were newly added which already had lots of platforms disabled. But after the merge, we just got more reports that these tests failed in the wild that lots of other platforms that are not in our CI. After some discussions, we will just disable the tests. atomic long double was and is broken. We need to fix the issue before we can have these tests running.
We cannot fix it in the library side by our own. We need clang’s support (at least provide a way to zero out the long double’s 56bits padding)
https://github.com/llvm/llvm-project/pull/74201
More information about the libcxx-commits
mailing list