[PATCH] D97676: [DSE] Extending isOverwrite to support offsetted fully overlapping stores

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D97676#2593922 <https://reviews.llvm.org/D97676#2593922>, @nikic wrote:

> Using scalar evolution for this purpose is definitely inappropriate. I would suggest to try basing this on D93529 <https://reviews.llvm.org/D93529> instead.

While using scalar evolution for the problem at hand is inappropriate, it might be worth a try to use SCEV to add 'loop-aware' DSE to catch cases like https://bugs.llvm.org/show_bug.cgi?id=46460. Just some food for thought in case anyone is interested in tackling this issue :)


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

https://reviews.llvm.org/D97676