[PATCH] D13363: [DeadStoreElimination] Add support for DSE across blocks

Ivan Baev via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 18 17:23:25 PDT 2015


ivanbaev updated the summary for this revision.
ivanbaev updated this revision to Diff 37712.
ivanbaev added a comment.

The new version now handles a redundant store in the if-block in if-then-else - based on a suggestion by Mitch. We use SafeBlocks set, which is initialized and updated in  DSE::HandleNonLocalDependency() and used in helper FindSafePreds(). SafeBlocks keeps track of all predecessor blocks that are safe to search through. A new test - ifthenelse.ll - has been added.


http://reviews.llvm.org/D13363

Files:
  lib/Transforms/Scalar/DeadStoreElimination.cpp
  test/Transforms/DeadStoreElimination/cycle.ll
  test/Transforms/DeadStoreElimination/ifthen.ll
  test/Transforms/DeadStoreElimination/ifthenelse.ll
  test/Transforms/DeadStoreElimination/loop.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13363.37712.patch
Type: text/x-patch
Size: 20659 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151019/b68a028a/attachment.bin>


More information about the llvm-commits mailing list