<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>LGTM. Small formatting suggestion:</div><div><br></div><div><div>+ //We would like to start over since some instructions are deleted</div><div>+ //and the iterator may become invalid value</div></div><div><br></div><div>Please insert spaces after the comment mark and end the sentence with a period. </div><div><br></div><div>Thanks,</div><div>Nadav</div><div><br></div><br><div><div>On Aug 19, 2013, at 4:53 PM, Yi Jiang <<a href="mailto:yjiang@apple.com">yjiang@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hi, <div><br></div><div>Please help to review the patch for PR16899. The reason of crash is that the instructions in the BB could be erased in the loop body and the iterator value is invalid. Thank Joerg for submitting a workaround on this in r188605. While basically in this patch we would like to update iterator "it" and "e" and start over going though the basic block every time we find that the BB changed, in this case we may find more potential opportunities without invalidate the iterators. To avoid redundant check, We also create a SmallSet to record the instructions we have visited and make sure we do not check them twice. Also the test case is included. Please see the patch for details and any comments are appreciated. Thank you. </div><div><br></div><div><pre style="white-space: pre-wrap; ">Files:</pre><div>lib/Transforms/Vectorize/SLPVectorizer.cpp</div></div><div>test/Transforms/SLPVectorizer/X86/pr16899.ll</div><div><br></div><div></div></div><span><pr16899.patch></span><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div></div></div>
_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br></blockquote></div><br></body></html>