<div dir="ltr">OK, I was just making sure. LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">2017-02-22 18:53 GMT+01:00 Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yes, but i don't expect to get to that for a month.<div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 22, 2017 at 9:25 AM, Piotr Padlewski via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Prazek added inline comments.<br>
<br>
<br>
================<br>
Comment at: include/llvm/Transforms/Utils/<wbr>MemorySSAUpdater.h:90-94<br>
+  // The below are utility functions. Other than creation of accesses to pass<br>
+  // to insertDef, and removeAccess to remove accesses, you should generally<br>
+  // not attempt to update memoryssa yourself. It is very non-trivial to get<br>
+  // the edge cases right, and the above calls already operate in near-optimal<br>
+  // time bounds.<br>
----------------<br>
The plan is to remove these functions in the future right?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D30221" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3022<wbr>1</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>