[PATCH] D105545: [MergedLoadStoreMotion] Conditional store elimination

[Momchil Velikov via Phabricator via llvm-commits]

chill updated this revision to Diff 357569.
chill edited the summary of this revision.
chill added a comment.
Herald added subscribers: asbirlea, george.burgess.iv.

Updated with a very different implementation. Now handles non-local stores as well.
Updated the description with a justification why I think this transformation is correct.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105545/new/

https://reviews.llvm.org/D105545

Files:
  llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
  llvm/test/CodeGen/AMDGPU/opt-pipeline.ll
  llvm/test/Other/opt-LTO-pipeline.ll
  llvm/test/Other/opt-O2-pipeline.ll
  llvm/test/Other/opt-O3-pipeline-enable-matrix.ll
  llvm/test/Other/opt-O3-pipeline.ll
  llvm/test/Other/opt-Os-pipeline.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105545.357569.patch
Type: text/x-patch
Size: 21642 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210709/c6e4f2bb/attachment.bin>