<div dir="ltr">LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 24, 2015 at 8:01 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>As discussed in `Interaction of target attribute change with modules`:</div><div><br></div><div>Ever since the target attributes change, we don't need to guard these headers with `requires`. Actually it's a bit worse, because if we do then they are included textually under the covers, causing declarations to appear in submodules they aren't supposed to be in.<br></div><div><br></div><div>Since it seems like only the x86 headers have been given the target attributes treatment, I've only removed those `requires`.</div><div><br></div><div>John, Paul, this affects our modules stuff internally, so CC'ing you guys. Since we don't ship all the headers, we will need to prune out of the module map any of the x86 headers that we don't ship, otherwise our packaged toolchain will encounter "header not found" errors when building the module.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div><br></div><div>-- Sean Silva</div></font></span></div>
</blockquote></div><br></div>