Emit an error when mixing <stdatomic.h> and <atomic>

Atomics in C and C++ are incompatible at the moment and mixing the
headers can result in confusing error messages.

Emit an error explicitly telling about the incompatibility. Introduce
the macro `__ALLOW_STDC_ATOMICS_IN_CXX__` that allows to choose in C++
between C atomics and C++ atomics.

rdar://problem/27435938

Reviewers: rsmith, EricWF, mclow.lists

Reviewed By: mclow.lists

Subscribers: jkorous-apple, christof, bumblebritches57, JonChesterfield, smeenai, cfe-commits

Differential Revision: https://reviews.llvm.org/D45470


git-svn-id: https://llvm.org/svn/llvm-project/libcxx/trunk@331379 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Volodymyr Sapsai
2018-05-02 17:56:45 +00:00
parent b110f38082
commit 8fc472d96a
2 changed files with 31 additions and 0 deletions

View File

@@ -0,0 +1,28 @@
//===----------------------------------------------------------------------===//
//
// The LLVM Compiler Infrastructure
//
// This file is dual licensed under the MIT and the University of Illinois Open
// Source Licenses. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
//
// UNSUPPORTED: libcpp-has-no-threads
//
// <atomic>
// Test that including <atomic> fails to compile when we want to use C atomics
// in C++ and have corresponding macro defined.
// MODULES_DEFINES: __ALLOW_STDC_ATOMICS_IN_CXX__
#ifndef __ALLOW_STDC_ATOMICS_IN_CXX__
#define __ALLOW_STDC_ATOMICS_IN_CXX__
#endif
#include <atomic>
// expected-error@atomic:* {{<stdatomic.h> is incompatible with the C++ standard library}}
int main()
{
}