<div dir="ltr">SGTM, thanks!<div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 14, 2017 at 12:31 PM, Geoff Berry 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">gberry added a comment.<br>
<br>
I'm going to go ahead with this version of the fix (with the comments addressed).  I implemented the stronger version (i.e. looking at MSSA memory accesses when checking doesNotAccessMemory, onlyReadsMemory, mayReadFromMemory, mayWriteMemory) and saw a total of 2 differences in the values returned from these checks across all of our benchmarks, and no change in code generation at all.  Given the amount of code churn to implement this and how close we are to release, I'm going to take the conservative route.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D35317" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D35317</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>