<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 22, 2016 at 1:16 PM, George Burgess IV <span dir="ltr"><<a href="mailto:george.burgess.iv@gmail.com" target="_blank">george.burgess.iv@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">george.burgess.iv added a comment.<br>
<br>
Drive-by comments about comments (insert "we need to go deeper" meme here)<br>
<br>
<br>
================<br>
Comment at: lib/Transforms/Scalar/MergedLoadStoreMotion.cpp:791<br>
@@ +790,3 @@<br>
+  // Walk all accesses in first block, put them into a hash table.<br>
+  // We can't just use a norm<br>
<span class="">+  for (auto &Access : *Succ0Accesses) {<br>
----------------<br>
</span>Potentially unfinished thought?<br>
<br></blockquote><div><br></div><div>Don't remember what it was, removed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: lib/Transforms/Scalar/MergedLoadStoreMotion.cpp:797<br>
@@ +796,3 @@<br>
+    Instruction *I = MU->getMemoryInst();<br>
+    // Only move non-simple (atomic, volatile) loads.<br>
+    LoadInst *Load0 = dyn_cast<LoadInst>(I);<br>
----------------<br>
Did you mean "only move simple (non-atomic, non-volatile)"?<br>
<br></blockquote><div>yes </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D8688" rel="noreferrer" target="_blank">http://reviews.llvm.org/D8688</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>