<div dir="ltr">Apologies, this was dead code left over from the first half of a refactoring I did. I've tidied it up and clarified matters with an assert in r204184. Sorry for any confusion.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Mar 18, 2014 at 2:38 PM, Liang Wang <span dir="ltr"><<a href="mailto:netcasper@gmail.com" target="_blank">netcasper@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
I'm studying loop vectorizer.  I don't understand the code yet.  But<br>
it looks not right to assign L->begin() to E.  Is it a typo?<br>
<br>
Thanks,<br>
Liang<br>
<br>
diff --git a/lib/Transforms/Vectorize/LoopVectorize.cpp<br>
b/lib/Transforms/Vectorize/LoopVectorize.cpp<br>
index 435c005..87b5d79 100644<br>
--- a/lib/Transforms/Vectorize/LoopVectorize.cpp<br>
+++ b/lib/Transforms/Vectorize/LoopVectorize.cpp<br>
@@ -1067,7 +1067,7 @@ struct LoopVectorize : public FunctionPass {<br>
     // We only handle inner loops, so if there are children just recurse.<br>
     if (!L->empty()) {<br>
       bool Changed = false;<br>
-      for (Loop::iterator I = L->begin(), E = L->begin(); I != E; ++I)<br>
+      for (Loop::iterator I = L->begin(), E = L->end(); I != E; ++I)<br>
         Changed |= processLoop(*I);<br>
       return Changed;<br>
     }<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br></div>