[PATCH] D42087: [DSE] Improve handling of noop stores exposed after dead interfering stores are removed

[Sam Conrad via Phabricator via llvm-commits]

sameconrad updated this revision to Diff 130822.
sameconrad added a comment.

Updated to remove unnecessary isDef/isClobber check, lambda is no longer necessary.

I did some benchmarking but unfortunately I couldn't find any noticeable differences in executable size from this patch.  At least, nothing in test-suite seemed to result in any changes to the compiled binaries with this patch.  It seems like the situation this is addressing is probably pretty rare.


https://reviews.llvm.org/D42087

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42087.130822.patch
Type: text/x-patch
Size: 4743 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180121/951b904a/attachment.bin>