On Sun, Jul 14, 2013 at 3:51 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com" target="_blank">eli.friedman@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Sat, Jul 13, 2013 at 10:41 PM, Richard Smith<br>
<<a href="mailto:richard-llvm@metafoo.co.uk">richard-llvm@metafoo.co.uk</a>> wrote:<br>
> Author: rsmith<br>
> Date: Sun Jul 14 00:41:45 2013<br>
> New Revision: 186275<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=186275&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=186275&view=rev</a><br>
> Log:<br>
> Add missing include guards into headers in lib/Headers. While it may appear<br>
> that these headers should not be included more than once, they are in fact<br>
> included twice when building our builtins module (in order for it to generate<br>
> submodules for them), and without this, any modular build enabling AVX and<br>
> including any builtin header fails.<br>
<br>
</div>Err, I don't think we're actually supposed to be building submodules<br>
for these... they can't be compiled independently anyway.</blockquote><div><br></div><div>Well... this change lets us support the likes of</div><div><br></div><div>  @import _Builtin_intrinsics.intel.avx;</div><div><br>
</div><div>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?</div></div>