[libc++] Improve REQUIRES for _ExtInt test on recent Clangs
authorLouis Dionne <ldionne@apple.com>
Tue, 1 Sep 2020 15:13:44 +0000 (11:13 -0400)
committerLouis Dionne <ldionne@apple.com>
Tue, 1 Sep 2020 16:03:06 +0000 (12:03 -0400)
The previous REQUIRES: would cause the test to run only on Clang-11, not
even on following versions of Clang, which was mostly not the intent.

libcxx/test/libcxx/atomics/ext-int.verify.cpp

index 3f57437..7cedcf1 100644 (file)
@@ -1,4 +1,19 @@
-// REQUIRES: clang-11
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+// <atomic>
+
+// Make sure that `std::atomic` doesn't work with `_ExtInt`. The intent is to
+// disable them for now until their behavior can be designed better later.
+// See https://reviews.llvm.org/D84049 for details.
+
+// UNSUPPORTED: clang-4, clang-5, clang-6, clang-7, clang-8, clang-9, clang-10
+// UNSUPPORTED: apple-clang-9, apple-clang-10, apple-clang-11, apple-clang-12
 
 #include <atomic>