On Sun, Jul 14, 2013 at 3:51 PM, Eli Friedman <[email protected]>wrote:
> On Sat, Jul 13, 2013 at 10:41 PM, Richard Smith > <[email protected]> wrote: > > Author: rsmith > > Date: Sun Jul 14 00:41:45 2013 > > New Revision: 186275 > > > > URL: http://llvm.org/viewvc/llvm-project?rev=186275&view=rev > > Log: > > Add missing include guards into headers in lib/Headers. While it may > appear > > that these headers should not be included more than once, they are in > fact > > included twice when building our builtins module (in order for it to > generate > > submodules for them), and without this, any modular build enabling AVX > and > > including any builtin header fails. > > Err, I don't think we're actually supposed to be building submodules > for these... they can't be compiled independently anyway. Well... this change lets us support the likes of @import _Builtin_intrinsics.intel.avx; which we already supported for the other submodules here. I'm not sure whether that was intentional, or whether this was just application of a "one submodule per .h file" rule. Doug?
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
