[PATCH] D19695: [MemorySSA] Fix bug in CachingMemorySSAWalker::getClobberingMemoryAccess.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 28 21:25:31 PDT 2016


So will you do a superset or should we approve this patch.

In retrospect, we should have added caching later with more full unit
tests, given how complex it ends up

On Thu, Apr 28, 2016, 4:51 PM George Burgess IV <george.burgess.iv at gmail.com>
wrote:

> george.burgess.iv added a comment.
>
> ...And it looks like this behavior already exists in the walker anyway
> (because we consult the cache first). Yay.
>
> I'll take a shot at fixing it, and add tests for cases like this.
>
>
> http://reviews.llvm.org/D19695
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160429/b2aa93e3/attachment.html>


More information about the llvm-commits mailing list