-Wextra-tokens would've caught this too -- I suppose it didn't fire since this is considered a system header by clang? Maybe libcxx's build process should include -Wsystem-headers?
On Sat, Feb 22, 2014 at 7:13 AM, Marshall Clow <[email protected]>wrote: > Author: marshall > Date: Sat Feb 22 09:13:48 2014 > New Revision: 201938 > > URL: http://llvm.org/viewvc/llvm-project?rev=201938&view=rev > Log: > Fix historical #ifdef. Use __cplusplus instead of > __GXX_EXPERIMENTAL_CXX0X__ when compiling with clang. No functionality > change. Noteto self: It is important to run the regression tests on the > copy of the code that you've changed. > > Modified: > libcxx/trunk/include/__config > > Modified: libcxx/trunk/include/__config > URL: > http://llvm.org/viewvc/llvm-project/libcxx/trunk/include/__config?rev=201938&r1=201937&r2=201938&view=diff > > ============================================================================== > --- libcxx/trunk/include/__config (original) > +++ libcxx/trunk/include/__config Sat Feb 22 09:13:48 2014 > @@ -215,7 +215,7 @@ > #define _LIBCPP_HAS_NO_TEMPLATE_ALIASES > #endif > > -#ifndef __GXX_EXPERIMENTAL_CXX0X__ > +#if __cplusplus < 201103L > #ifdef __linux__ > #define _LIBCPP_HAS_NO_UNICODE_CHARS > #else > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
