[PATCH] D102096: [DAGCombiner] Fix DAG combine store elimination, different address space.

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D102096#2747951 <https://reviews.llvm.org/D102096#2747951>, @hgreving wrote:

> In D102096#2747115 <https://reviews.llvm.org/D102096#2747115>, @xbolva00 wrote:
>
>> What about DeadStoreElimination ? It can also merge stores.. maybe dse needs similar fix.
>
> https://www.godbolt.org/z/aqncWfb4c seems to suggest maybe, though haven't debugged whether it's DSE or DAG combine again who does it.

If you show the IR for that example, it looks ok:
https://www.godbolt.org/z/55v7anfa3
I don't see any explicit references to address spaces within DSE itself, so I assume the underlying ValueTracking and/or Alias Analysis is ok here.


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

https://reviews.llvm.org/D102096