<div dir="ltr">You can get rid of UseInst too, MemoryLocOrCall will take a MemoryUse directly :)<div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 12, 2016 at 8:20 PM, Sebastian Pop <span dir="ltr"><<a href="mailto:sebpop@gmail.com" target="_blank">sebpop@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Oct 12, 2016 at 10:05 PM, Daniel Berlin <<a href="mailto:dberlin@dberlin.org">dberlin@dberlin.org</a>> wrote:<br>
> There is actually a much simpler fix you sholdl use.<br>
><br>
> return instructionClobbersQuery(MD, MU MemoryLocOrCall(MU), AA);<br>
><br>
> You need to move that version of instructionClobbersQuery up, but it was<br>
> built to always do the right thing.<br>
><br>
> Also note it will properly handle a case you are not: Fences do not have<br>
> memorylocations, and calling get on them will also crash.<br>
><br>
<br>
</span>Right. I will commit the attached patch after testing.<br>
<br>
Thanks,<br>
Sebastian<br>
</blockquote></div><br></div>