[PATCH] D84049: Disable use of _ExtInt with '__atomic' builtins

ldionne added inline comments. Comment at: libcxx/test/libcxx/atomics/ext-int.verify.cpp:1 +// REQUIRES: clang-11 + jtmott-intel wrote: > ldionne wrote: > > This isn't great, since it won't run on clang-12, etc. I'll change it to: > > > > ``` > > //

[PATCH] D84049: Disable use of _ExtInt with '__atomic' builtins

jtmott-intel added inline comments. Comment at: libcxx/test/libcxx/atomics/ext-int.verify.cpp:1 +// REQUIRES: clang-11 + ldionne wrote: > This isn't great, since it won't run on clang-12, etc. I'll change it to: > > ``` > // UNSUPPORTED: clang-4, clang-5,

[PATCH] D84049: Disable use of _ExtInt with '__atomic' builtins

ldionne added inline comments. Comment at: libcxx/test/libcxx/atomics/ext-int.verify.cpp:1 +// REQUIRES: clang-11 + This isn't great, since it won't run on clang-12, etc. I'll change it to: ``` // UNSUPPORTED: clang-4, clang-5, clang-6, clang-7, clang-8,

[PATCH] D84049: Disable use of _ExtInt with '__atomic' builtins

hans added a comment. In D84049#2224061 , @jtmott-intel wrote: > Committed to master, and created bug to cherry pick into 11.0. > > https://bugs.llvm.org/show_bug.cgi?id=47222 Pushed to 11.x as 82e48a579024d0ffbc352702ec0c52b47a6fe691

[PATCH] D84049: Disable use of _ExtInt with '__atomic' builtins

jtmott-intel added a comment. Committed to master, and created bug to cherry pick into 11.0. https://bugs.llvm.org/show_bug.cgi?id=47222 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84049/new/ https://reviews.llvm.org/D84049

[PATCH] D84049: Disable use of _ExtInt with '__atomic' builtins

This revision was automatically updated to reflect the committed changes. Closed by commit rGca77ab494aa2: Disable use of _ExtInt with __atomic builtins (authored by jtmott-intel). Herald added projects: clang, libc++. Herald added subscribers: libcxx-commits, cfe-commits. Herald added a