[libcxx] r333327 - Fix GCC handling of ATOMIC_VAR_INIT

JF Bastien via cfe-commits cfe-commits at lists.llvm.org
Fri May 25 17:13:53 PDT 2018


Author: jfb
Date: Fri May 25 17:13:53 2018
New Revision: 333327

URL: http://llvm.org/viewvc/llvm-project?rev=333327&view=rev
Log:
Fix GCC handling of ATOMIC_VAR_INIT

r333325 from D47225 added warning checks, and the test was written to be C++11 correct by using ATOMIC_VAR_INIT (note that the committee fixed that recently...). It seems like GCC can't handle ATOMIC_VAR_INIT well because it generates 'type 'std::atomic<int>' cannot be initialized with an initializer list' on bot libcxx-libcxxabi-x86_64-linux-ubuntu-cxx03. Drop the ATOMIC_VAR_INITs since they weren't required to test the diagnostics.

Modified:
    libcxx/trunk/test/libcxx/atomics/diagnose_nonnull.fail.cpp

Modified: libcxx/trunk/test/libcxx/atomics/diagnose_nonnull.fail.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/libcxx/atomics/diagnose_nonnull.fail.cpp?rev=333327&r1=333326&r2=333327&view=diff
==============================================================================
--- libcxx/trunk/test/libcxx/atomics/diagnose_nonnull.fail.cpp (original)
+++ libcxx/trunk/test/libcxx/atomics/diagnose_nonnull.fail.cpp Fri May 25 17:13:53 2018
@@ -17,8 +17,8 @@
 #include <atomic>
 
 int main() {
-  std::atomic<int> ai = ATOMIC_VAR_INIT(0);
-  volatile std::atomic<int> vai = ATOMIC_VAR_INIT(0);
+  std::atomic<int> ai;
+  volatile std::atomic<int> vai;
   int i = 42;
 
   atomic_is_lock_free((const volatile std::atomic<int>*)0); // expected-error {{null passed to a callee that requires a non-null argument}}




More information about the cfe-commits mailing list