<div dir="ltr">Sorry, forgot about this :-(  Here's the patched to release_35 which removes reference to unroll-related pragmas and the respective metadata as discussed earlier in the thread.  These are doc-only (*.rst) patches.  OK?<div><br></div><div>Bill, can these be merged to the release branch?  Background: 3.5 was cut in the middle of a number of changes to support the unroll pragma.  Since the cut, the metadata and unroll pragma syntax have changed and a couple of small bug fixes went in.  These doc patches remove any reference to the unroll pragma and metadata so we don't have to carry forward support for the older syntax.  Also, this avoid publicizing the unroll pragmas which at the time of the cut had a couple of unpatched bugs.</div><div><br><div>Thanks!</div><div>Mark</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 25, 2014 at 2:39 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">----- Original Message -----<br>
> From: "Mark Heffernan" <<a href="mailto:meheff@google.com">meheff@google.com</a>><br>
> To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>><br>
> Cc: "Eric Christopher" <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>>, "cfe-commits" <<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a>>, <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>,<br>
> "Aaron Ballman" <<a href="mailto:aaron.ballman@gmail.com">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>
</span><span class="">> On Tue, Aug 19, 2014 at 5:42 PM, Hal Finkel < <a href="mailto:hfinkel@anl.gov">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>
</span>If it is not documented nor officially supported, probably not.<br>
<span class="HOEnZb"><font color="#888888"><br>
 -Hal<br>
</font></span><div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div>