<div dir="ltr">This is what I get for skimming until the last comment :(<div><br></div><div>Sorry Adam, I'll make that change first thing in the morning.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, 2 Sep 2015 at 18:32 Adam Nemet via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">anemet added inline comments.<br>
<br>
================<br>
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:2617-2618<br>
@@ +2616,4 @@<br>
+  // Find the loop boundaries.<br>
+  const SCEV *ExitCount = SE->getBackedgeTakenCount(OrigLoop);<br>
+  assert(ExitCount != SE->getCouldNotCompute() && "Invalid loop count");<br>
+<br>
----------------<br>
anemet wrote:<br>
> I'll let Michael give final OK but since you're cleaning this up, it would be good to finally rename this to BackedgeTakenCount or something.  Later the code redefines this same variable to the *real* exit count which is pretty crazy.  These two values should be different variables.<br>
Looks like you didn't make this change in the commit.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D12475" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12475</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>