[PATCH] D19676: [MemorySSA] Special case for assume intrinsics.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 28 13:00:18 PDT 2016


It seems like we should just fix that?
I see why it would do that, but it seems it doesn't separate whether it can
sink thinks with whether it should :)

(Note: there should be no other users, AFAIK, since i added this version of
getModRefInfo)



On Thu, Apr 28, 2016 at 12:41 PM, Geoff Berry <gberry at codeaurora.org> wrote:

> gberry added a comment.
>
> 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.
>
>
> http://reviews.llvm.org/D19676
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160428/eb186f1b/attachment.html>


More information about the llvm-commits mailing list