<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 22, 2016 at 5:02 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hfinkel added a subscriber: hfinkel.<br>
<br>
================<br>
Comment at: llvm/lib/Transforms/Scalar/<wbr>GVNHoist.cpp:368<br>
@@ -363,1 +367,3 @@<br>
<span class="">+<br>
+      if (!AA->isNoAlias(DefLoc, MemoryLocation::get(Insn)))<br>
         return true;<br>
</span>----------------<br>
You could do something like:<br>
<br>
  if (AA->getModRef(Insn, DefLoc) != MRI_NoModRef)<br>
    return true;<br>
<br>
and that should handle calls.<br>
<br></blockquote><div><br></div><div>(This will only work if MemoryDef is really always a StoreInst)</div><div><br></div></div></div></div>