[PATCH] D11854: DeadStoreElimination: remove a redundant store even if the load is in a different block.

[Erik Eckstein via llvm-commits]

eeckstein updated this revision to Diff 31751.
eeckstein added a comment.

This updated version uses a work-list algorithm to do the safety checking. This has 2 advantages compared to MD->getNonLocalPointerDependency:

1. It ignores load instructions which should not prevent the optimization
2. It's faster because it bails out at the first memory alias it finds.

I still have to do some measurements. I will post some data in the next time.


http://reviews.llvm.org/D11854

Files:
  lib/Transforms/Scalar/DeadStoreElimination.cpp
  test/Transforms/DeadStoreElimination/simple.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11854.31751.patch
Type: text/x-patch
Size: 6507 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150811/1de2a52a/attachment.bin>