<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 19, 2015 at 5:03 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Do you want to add #pragma unroll back to the release notes for 3.6?<br></blockquote><div><br></div><div>Thanks for the reminder!  I went ahead and submitted the change (r230294).  It is virtually identical to what was approved for the 3.5 branch.  I just added a bit about being able to equivalently express the unroll hints using "#pragma clang loop" form.</div><div><br></div><div>Thanks!<br></div><div>Mark<br></div><div> </div><!--
--><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span><font color="#888888"><br>
 - Hans<br>
</font></span><div><div><br>
On Tue, Sep 2, 2014 at 10:43 AM, Mark Heffernan <<a href="mailto:meheff@google.com" target="_blank">meheff@google.com</a>> wrote:<br>
> And here are the patches...<br>
><br>
> Mark<br>
><br>
><br>
><br>
> On Tue, Sep 2, 2014 at 10:42 AM, Mark Heffernan <<a href="mailto:meheff@google.com" target="_blank">meheff@google.com</a>> wrote:<br>
>><br>
>> Sorry, forgot about this :-(  Here's the patched to release_35 which<br>
>> removes reference to unroll-related pragmas and the respective metadata as<br>
>> discussed earlier in the thread.  These are doc-only (*.rst) patches.  OK?<br>
>><br>
>> Bill, can these be merged to the release branch?  Background: 3.5 was cut<br>
>> in the middle of a number of changes to support the unroll pragma.  Since<br>
>> the cut, the metadata and unroll pragma syntax have changed and a couple of<br>
>> small bug fixes went in.  These doc patches remove any reference to the<br>
>> unroll pragma and metadata so we don't have to carry forward support for the<br>
>> older syntax.  Also, this avoid publicizing the unroll pragmas which at the<br>
>> time of the cut had a couple of unpatched bugs.<br>
>><br>
>> Thanks!<br>
>> Mark<br>
>><br>
>><br>
>> On Mon, Aug 25, 2014 at 2:39 PM, Hal Finkel <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>> wrote:<br>
>>><br>
>>> ----- Original Message -----<br>
>>> > From: "Mark Heffernan" <<a href="mailto:meheff@google.com" target="_blank">meheff@google.com</a>><br>
>>> > To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>><br>
>>> > Cc: "Eric Christopher" <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>>, "cfe-commits"<br>
>>> > <<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a>>, <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a>,<br>
>>> > "Aaron Ballman" <<a href="mailto:aaron.ballman@gmail.com" target="_blank">aaron.ballman@gmail.com</a>><br>
>>> > Sent: Thursday, August 21, 2014 6:22:16 PM<br>
>>> > Subject: Re: release_35 patches for unroll pragma<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > On Tue, Aug 19, 2014 at 5:42 PM, Hal Finkel < <a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a> ><br>
>>> > wrote:<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > Unfortunately, I was on vacation that week, and I'm still playing<br>
>>> > catch-up on my e-mail...<br>
>>> ><br>
>>> > At this point I think it is too late to pull in these kinds of<br>
>>> > changes, but, if we didn't previously, we should add auto-upgrade<br>
>>> > support for the renamed metadata.<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > Recently got back from vacation myself... Earlier in the thread there<br>
>>> > was the suggestion of removing mention of the unroll pragma and<br>
>>> > metadata from the 3.5 docs because support for it is incomplete. I<br>
>>> > agree and I'll send out a patch for this. Would it still be<br>
>>> > necessary to add the auto-upgrade support since the unroll pragma<br>
>>> > and associated metadata would not be officially supported?<br>
>>><br>
>>> If it is not documented nor officially supported, probably not.<br>
>>><br>
>>>  -Hal<br>
>>><br>
>>> ><br>
>>> ><br>
>>> > Thanks,<br>
>>> > Mark<br>
>>> ><br>
>>> ><br>
>>> > -Hal<br>
>>> ><br>
>>> > ><br>
>>> > > -eric<br>
>>> > ><br>
>>> ><br>
>>> > --<br>
>>> > Hal Finkel<br>
>>> > Assistant Computational Scientist<br>
>>> > Leadership Computing Facility<br>
>>> > Argonne National Laboratory<br>
>>> ><br>
>>> ><br>
>>><br>
>>> --<br>
>>> Hal Finkel<br>
>>> Assistant Computational Scientist<br>
>>> Leadership Computing Facility<br>
>>> Argonne National Laboratory<br>
>><br>
>><br>
><br>
><br>
</div></div><div><div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div></blockquote></div><br></div></div>