<div dir="ltr">...I need to remember to refresh my browser before reviewing things. Ignore my "IIRC, ..." comment, please. :)<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 2, 2016 at 2:51 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 accepted this revision.<br>
george.burgess.iv added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM with nits. Thanks!<br>
<br>
<br>
================<br>
Comment at: lib/Transforms/Utils/MemorySSA.cpp:141<br>
@@ +140,3 @@<br>
+      return CS.getCalledValue() == Other.CS.getCalledValue();<br>
+    else<br>
+      return Loc == Other.Loc;<br>
----------------<br>
IIRC, this else was removed in a post-commit review? (If not, please remove it anyway :) )<br>
<br>
================<br>
Comment at: lib/Transforms/Utils/MemorySSA.cpp:196<br>
@@ +195,3 @@<br>
+                                    AA);<br>
+  else<br>
+    return instructionClobbersQuery(MD, UseMLOC.getLoc(), MU->getMemoryInst(),<br>
----------------<br>
Please remove `else`<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23072" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23072</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>