[cfe-commits] r154499 - in /cfe/trunk: lib/Sema/SemaOverload.cpp test/CodeGenCXX/atomicinit.cpp

[Chandler Carruth]

On Wed, Apr 11, 2012 at 4:51 PM, David Chisnall <csdavec at swan.ac.uk> wrote:

> On 11 Apr 2012, at 16:46, Chandler Carruth wrote:
>
> > It seems like this should get an ext-warn...
>
> In which case we'd get a load of them for anything using libc++.
>

No? System headers suppress warnings fer exactly this reason.


> > Is there not an existing test for _Atomic as used in C++ that this could
> be folded into?
>
> $ grep _Atomic test/CodeGenCXX/*
> atomicinit.cpp:  _Atomic(int) i;
>
> So, no.  This test should be growing a bit soon, once I fix the issue with
> __atomic_init() with C++ types.
>

Ahh, maybe rename it to c11_atomics.cpp or something to clarify that its
for testing the interaction between C++ and C11 atomics? Anyways, just
trying to keep the growth of test files reasonable.

Thanks!
-Chandler


>
> David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120411/a1cd6011/attachment.html>