[PATCH] D11143: [RFC] Cross Block DSE

[Karthik Bhat]

karthikthecool updated this revision to Diff 30807.
karthikthecool added a comment.

Hi All,
Update the patch to fix a failure in LLVM-LNT testcase on 32bit ubuntu. Unfortunetly i was not able to catch it on my 64bit machine.
This failure occured as we incorrectly deleted a store that was followed by a store that partially aliased it.
When seraching if a candidate store is safe we conservately mark a store as unsafe if we found a store that may/partially alias the candidate store.
This fixes failure found in LLVM LNT on 32bit ubuntu. 
We still observe some 30%+ execution time improvement in MultiSource/Benchmarks/Trimaran/enc-pc1/enc-pc1.

Thanks and Regards
Karthik Bhat


http://reviews.llvm.org/D11143

Files:
  lib/Transforms/Scalar/DeadStoreElimination.cpp
  test/Transforms/DeadStoreElimination/cross_block_dse.ll
  test/Transforms/DeadStoreElimination/cross_block_dse_loop.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11143.30807.patch
Type: text/x-patch
Size: 19502 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150728/c09cf6db/attachment.bin>