<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 20, 2016 at 12:34 PM, Geoff Berry <span dir="ltr"><<a href="mailto:gberry@codeaurora.org" target="_blank">gberry@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">gberry added a comment.<br>
<br>
Would it make sense to split out the DFS numbering code motion barrier code into a separate change?<br>
<br></blockquote><div><br></div><div>I'm happy to commit it separately if anyone cares.</div><div>I'll be honest; I don't actually  care enough about the pass to want to speed it up outside of memoryssa.</div><div>While the DFS numbering removes an N^3'ness added to the existing code, the code is N^3 in so many places it's hard to keep track :)</div><div>On the other hand, it's necessary  for memoryssa because memoryssa does not normally walk the blocks at all</div><div><br></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:977<br>
@@ +976,3 @@<br>
+///<br>
+/// \brief True when the store is not downwards explosed to block \p End<br>
+<br>
----------------<br>
Typo: "explosed".<br>
Also, there is no 'End' parameter as referenced in the comment.<br>
<br>
<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>