[PATCH] D8688: Update MergedLoadStoreMotion to use MemorySSA

[Daniel Berlin via llvm-commits]

This version should correctly update memoryssa for store sinking :)
(I checked that it does on the llvm testcases, i'll test it further as i
split out the memoryssa part)

On Wed, Jun 15, 2016 at 11:49 AM, Daniel Berlin <dberlin at dberlin.org> wrote:

> dberlin updated this revision to Diff 60871.
> dberlin added a comment.
>
> - Add verification for completeness of PHI nodes
> - Add basic PHI creation API
> - Update MemorySSA for store sinking
>
>
> http://reviews.llvm.org/D8688
>
> Files:
>   include/llvm/Transforms/Utils/MemorySSA.h
>   lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
>   lib/Transforms/Utils/MemorySSA.cpp
>   test/Transforms/InstMerge/exceptions.ll
>   test/Transforms/InstMerge/ld_hoist1.ll
>   test/Transforms/InstMerge/ld_hoist_st_sink.ll
>   test/Transforms/InstMerge/st_sink_barrier_call.ll
>   test/Transforms/InstMerge/st_sink_bugfix_22613.ll
>   test/Transforms/InstMerge/st_sink_no_barrier_call.ll
>   test/Transforms/InstMerge/st_sink_no_barrier_load.ll
>   test/Transforms/InstMerge/st_sink_no_barrier_store.ll
>   test/Transforms/InstMerge/st_sink_two_stores.ll
>   test/Transforms/InstMerge/st_sink_with_barrier.ll
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160615/269a30b5/attachment.html>