<div dir="ltr">Great, thanks.  Submitted.<div>Clang: r213587<div>LLVM: r213588</div></div><div><br></div><div>Mark</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 21, 2014 at 2:16 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"><div class="">----- Original Message -----<br>
> From: "Arnold Schwaighofer" <<a href="mailto:aschwaighofer@apple.com">aschwaighofer@apple.com</a>><br>
> To: "Mark Heffernan" <<a href="mailto:meheff@google.com">meheff@google.com</a>><br>
> Cc: "Tyler Nowicki" <<a href="mailto:tnowicki@apple.com">tnowicki@apple.com</a>>, "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>>, "nadav" <<a href="mailto:nadav@apple.com">nadav@apple.com</a>>, "llvm-commits"<br>

> <<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>>, "llvm cfe" <<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a>><br>
</div><div class="">> Sent: Monday, July 21, 2014 4:13:36 PM<br>
> Subject: Re: Rename metadata llvm.loop.vectorize.unroll to llvm.loop.vectorize.interleave<br>
><br>
> The llvm patch LGTM.<br>
><br>
><br>
> The clang patch seems obvious, too.<br>
><br>
><br>
<br>
</div>I agree. Both patches are good to commit.<br>
<span class="HOEnZb"><font color="#888888"><br>
 -Hal<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
><br>
><br>
> On Jul 21, 2014, at 2:00 PM, Mark Heffernan < <a href="mailto:meheff@google.com">meheff@google.com</a> ><br>
> wrote:<br>
><br>
><br>
> Any comments this renaming patch? It'd be good to get this in before<br>
> the cut. Thanks!<br>
><br>
><br>
> Mark<br>
><br>
><br>
><br>
> On Fri, Jul 18, 2014 at 4:13 PM, Mark Heffernan < <a href="mailto:meheff@google.com">meheff@google.com</a> ><br>
> wrote:<br>
><br>
><br>
><br>
><br>
><br>
><br>
> On Fri, Jul 18, 2014 at 4:08 PM, Tyler Nowicki < <a href="mailto:tnowicki@apple.com">tnowicki@apple.com</a> ><br>
> wrote:<br>
><br>
><br>
><br>
><br>
> Looks like the llvm patch was accidentally overwritten by the clang<br>
> patch.<br>
><br>
><br>
> Whoops. Sorry. Here are both patches again.<br>
><br>
><br>
><br>
><br>
> Mark<br>
><br>
><br>
><br>
><br>
><br>
><br>
> Tyler<br>
><br>
><br>
><br>
><br>
><br>
> On Jul 18, 2014, at 4:03 PM, Mark Heffernan < <a href="mailto:meheff@google.com">meheff@google.com</a> ><br>
> wrote:<br>
><br>
><br>
><br>
><br>
> Here's the updated patches. llvm.loop.vectorize.interleave is now<br>
> llvm.loop.interleave.count.<br>
><br>
><br>
> Mark<br>
><br>
><br>
> On Fri, Jul 18, 2014 at 2:25 PM, Mark Heffernan < <a href="mailto:meheff@google.com">meheff@google.com</a> ><br>
> wrote:<br>
><br>
><br>
><br>
><br>
><br>
><br>
> On Fri, Jul 18, 2014 at 2:11 PM, Tyler Nowicki < <a href="mailto:tnowicki@apple.com">tnowicki@apple.com</a> ><br>
> wrote:<br>
><br>
><br>
> Perhaps you could rename ‘loop.vectorize.unroll' to<br>
> ‘loop.interleave.count'. That way in a future patch we could add<br>
> ‘loop.interleave.enable'.<br>
><br>
> What do you think?<br>
><br>
><br>
><br>
> Sounds reasonable. Happy to change it to whatever is most<br>
> appropriate.<br>
><br>
><br>
> Mark<br>
><br>
><br>
><br>
><br>
><br>
> Tyler<br>
><br>
><br>
> On Jul 18, 2014, at 1:51 PM, Hal Finkel < <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> > wrote:<br>
><br>
> > Nadav, Arnold,<br>
> ><br>
> > I think this makes sense. Do either of you object?<br>
> ><br>
> > -Hal<br>
> ><br>
> > ----- Original Message -----<br>
> >> From: "Mark Heffernan" < <a href="mailto:meheff@google.com">meheff@google.com</a> ><br>
</div></div><div class="HOEnZb"><div class="h5">> >> To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a> , "cfe-commits" <<br>
> >> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a> >, "Hal Finkel" < <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> >,<br>
> >> "Tyler<br>
> >> Nowicki" < <a href="mailto:tnowicki@apple.com">tnowicki@apple.com</a> ><br>
> >> Sent: Friday, July 18, 2014 3:44:00 PM<br>
> >> Subject: Rename metadata llvm.loop.vectorize.unroll to<br>
> >> llvm.loop.vectorize.interleave<br>
> >><br>
> >><br>
> >> This patch rename metadata llvm.loop.vectorize.unroll to<br>
> >> llvm.loop.vectorize.interleave to avoid confusion with the<br>
> >> concatenation unroller metadata (llvm.loop.unroll). It fell out of<br>
> >> a<br>
> >> discussion on patch <a href="http://reviews.llvm.org/D4576" target="_blank">http://reviews.llvm.org/D4576</a> . On that patch<br>
> >> there was some confusion about whether llvm.loop.vectorize.unroll<br>
> >> controlled the traditional (concatenation) loop unroller. And IIRC<br>
> >> this isn't the first time there has been this confusion. Renaming<br>
> >> vectorize.unroll to vectorize.interleave also more closely matches<br>
> >> the pragma which generates the metadata (#pragma clang loop<br>
> >> interleave_count). There are still lots of uses of "unroll" within<br>
> >> the code in function/variable names and comments. I have no<br>
> >> intention of changing those.<br>
> >><br>
> >><br>
> >> Mark<br>
> ><br>
> > --<br>
> > Hal Finkel<br>
> > Assistant Computational Scientist<br>
> > Leadership Computing Facility<br>
> > Argonne National Laboratory<br>
><br>
><br>
><br>
> <rename.llvm.patch> <rename.clang.patch><br>
><br>
><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>