<div dir="ltr">Ping?  Hal or Aaron?  Is it too late to get these in?  I was on vacation last week so wasn't able to follow up earlier on this.<div><br></div><div>Thanks,<div>Mark</div></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Jul 29, 2014 at 12:35 PM, Mark Heffernan <span dir="ltr"><<a href="mailto:meheff@google.com" target="_blank">meheff@google.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>Hi,</div><div><br></div><div>Below are a handful of patches I submitted last week for the loop unroll pragma which would be nice to get into 3.5.  The bug fix and metadata renaming ones I'd particularly like to see get in.  The "#pragma nounroll" is more of a nice-to-have.  All of the patches should only affect code which uses one of the forms of the loop unroll pragmas.</div>


<div><br></div><div>Rename metadata.  In unroll pragma syntax and loop hint metadata, change "enable" forms to a new form using the string "full":<br></div><div><div>r213771 (clang)</div><div>r213772 (llvm)</div>


<div>r213775 (fix broken test caused by r213771)</div><div><br></div><div>Add support for #pragma nounroll :</div><div>r213885</div><div><br></div><div>Bug fixes:</div><div>r213789 : Do not add unroll disable metadata after unrolling pass for loops with #pragma clang loop unroll(full).</div>


<div>r213900 : Fix bug where some loops with loop unroll_count(N) would mistakenly not be unrolled.</div></div><div><br></div><div>Let me know if these are reasonable.</div><div><br></div><div>Thanks!</div><span class="HOEnZb"><font color="#888888"><div>
Mark</div>

</font></span></div>
</blockquote></div><br></div>