<div dir="ltr">I responded to all the comments via Phabricator over an hour ago, but Phabricator never sent out the corresponding email :-(  I've verified that others can see my comment at <a href="http://reviews.llvm.org/D4090" target="_blank">http://reviews.llvm.org/D4090</a> so please look there for my responses and latest patch.<div>

<br></div><div>Mark</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 10, 2014 at 4:02 PM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: lib/Transforms/Scalar/LoopUnrollPass.cpp:274<br>
@@ +273,3 @@<br>
+    bool HasEnablePragma = HasUnrollEnablePragma(L, Enable);<br>
+    int PragmaCount;<br>
+    bool HasCountPragma = HasUnrollCountPragma(L, PragmaCount);<br>
----------------<br>
Move this closer to use<br>
<br>
================<br>
Comment at: test/Transforms/LoopUnroll/unroll-pragmas.ll:48<br>
@@ +47,3 @@<br>
+  %exitcond = icmp eq i64 %indvars.iv.next, 4<br>
+  br i1 %exitcond, label %for.end, label %for.body, !llvm.loop !1<br>
+<br>
----------------<br>
IMHO it would be more readable if you placed !1 right after this function. I don't think the mdnodes have to be necessarily collected in the bottom. Ditto for the rest<br>
<br>
<a href="http://reviews.llvm.org/D4090" target="_blank">http://reviews.llvm.org/D4090</a><br>
<br>
<br>
</blockquote></div><br></div>