<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 30, 2015 at 11:27 AM Vedant Kumar <<a href="mailto:vsk@apple.com">vsk@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> On Jul 30, 2015, at 10:33 AM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
><br>
>> I don't see any downsides to reintroducing these guards.<br>
><br>
> Then you weren't really paying attention to the point of removing them :)<br>
><br>
> The idea is so that the headers can be used, with appropriate target attributes, for any code.<br>
<br>
Right, I thought about this but wasn't sure if there were benefits to having symbols available for an unsupported target.<br>
<br>
I.e, is there some reason a project might want to include the header for SSE4 intrinsics if it can't use any of those symbols?<br>
<br></blockquote><div><br></div><div>I put a code snippet for something to do in the commit, but the general idea is that you can compile a function for a specific target with subtarget features and use the target attribute to add subtarget features and you'll want to be able to use the intrinsics at the same time. It won't work if you block them at the preprocessor level.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I'm just not 100% convinced that removing the header guards was necessary (which, I admit, could just be due to a lack of understanding on my part).<br><br></blockquote><div><br></div><div>Did the above help?</div><div><br></div><div>-eric </div></div></div>