[PATCH] D47225: Add nonnull; use it for atomics

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 25 17:15:30 PDT 2018


jfb added a comment.

GCC in  libcxx-libcxxabi-x86_64-linux-ubuntu-cxx03  seems to mis-handle ATOMIC_VAR_INIT:

  File /home/llvm-builder/llvm-buildslave-root/libcxx-libcxxabi-x86_64-linux-ubuntu-cxx03/llvm/projects/libcxx/test/libcxx/atomics/diagnose_nonnull.fail.cpp Line 20: non-aggregate type 'std::atomic<int>' cannot be initialized with an initializer list
  File /home/llvm-builder/llvm-buildslave-root/libcxx-libcxxabi-x86_64-linux-ubuntu-cxx03/llvm/projects/libcxx/test/libcxx/atomics/diagnose_nonnull.fail.cpp Line 21: non-aggregate type 'volatile std::atomic<int>' cannot be initialized with an initializer list

I'll drop the initialization for now, it's not required anyways.


Repository:
  rL LLVM

https://reviews.llvm.org/D47225





More information about the cfe-commits mailing list