[libcxx-commits] [PATCH] D154246: [libc++] Add check for building with picolibc

Michael Platings via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jul 18 02:37:51 PDT 2023


michaelplatings added inline comments.


================
Comment at: libcxx/test/std/atomics/atomics.types.generic/integral.pass.cpp:9
 
+// XFAIL: !atomics
+
----------------
michaelplatings wrote:
> philnik wrote:
> > I would just go with `// REQUIRES: atomics`. That's a lot less confusing that the double negation.
> I went with `XFAIL` for consistency with `non-lockfree-atomics`. `REQUIRES` means the tests won't be run at all, but I'm OK with that.
The [[ https://reviews.llvm.org/B246016 | CI results with XFAIL: !atomics ]] suggest that the code to detect atomics support was incorrect. REQUIRES wouldn't have picked that up so I feel XFAIL is the better choice.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154246/new/

https://reviews.llvm.org/D154246



More information about the libcxx-commits mailing list