<div dir="ltr">It seems like we should just fix that?<br><div>I see why it would do that, but it seems it doesn't separate whether it can sink thinks with whether it should :)</div><div><br></div><div>(Note: there should be no other users, AFAIK, since i added this version of getModRefInfo)</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 28, 2016 at 12:41 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>
I looked at that briefly, and I think that would have additional impact that we may not want.  For example, it looked to me like changing getModRefInfo to ignore llvm.assume would cause LICM to hoist/sink assumes which seems at first glance to potentially be bad.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D19676" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19676</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>