<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 4, 2014 at 10:35 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron.ballman@gmail.com" target="_blank">aaron.ballman@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 class="">On Mon, Aug 4, 2014 at 1:33 PM, Mark Heffernan <<a href="mailto:meheff@google.com">meheff@google.com</a>> wrote:<br>

> On Mon, Aug 4, 2014 at 10:24 AM, Aaron Ballman <<a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On Mon, Aug 4, 2014 at 1:22 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>><br>
>> wrote:<br>
>> > Any reason that we need them in 3.5? Correctness?<br>
>><br>
>> My only concern is that the feature is partially in 3.5, but a<br>
>> user-facing part of that feature was changed once the freeze happened.<br>
>> Eg)  #pragma clang loop unroll(enable) became  #pragma clang loop<br>
>> unroll(full)<br>
><br>
><br>
> That's my primary concern as well.  Having one release with one particular<br>
> syntax, then switch it to something else for the next release is not great.<br>
> All-in-all I'd probably prefer not supporting the unroll pragma at all in<br>
> 3.5 than have a (slightly) buggy one whose syntax will change.  However,<br>
> rolling back support completely would be a bigger change than these patches.<br>
<br>
</div>An alternate option would be to update the documentation to remove<br>
mention of the feature. That's a much smaller change. ;-)<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">If having a stealth feature like that is reasonable, I'm happy to remove mention of it from the docs.  More specifically any mention of the following would be removed: '#pragma unroll', '#pragma clang loop unroll', '#pragma clang loop unroll_count', and 'llvm.loop.unroll.*' metadata.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Mark</div></div>