[PATCH] D45470: Emit an error when mixing <stdatomic.h> and <atomic>
Volodymyr Sapsai via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 24 12:15:15 PDT 2018
vsapsai planned changes to this revision.
vsapsai added inline comments.
================
Comment at: clang/test/Headers/stdatomic.cpp:4
+
+#include <stdatomic.h>
+
----------------
mclow.lists wrote:
> Is there a reason we want to test this twice - once in clang and once in libc++?
> We can use `expected-error` in libc++ tests to check the error.
>
The idea was to test `<stdatomic.h>` in clang and `<atomic>` in libc++. Does that answer your question? I'm not sure I understood it correctly.
Good suggestion to use `expected-error` in libc++ test, will add that.
https://reviews.llvm.org/D45470
More information about the cfe-commits
mailing list