[PATCH] D11621: AMDGPU/SI: Mark SMRD instructions as rematerializable

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D11621#3958647 <https://reviews.llvm.org/D11621#3958647>, @rampitec wrote:

> I do not think this is OK. We may issue a scalar load before a first potentially aliasing store. After the store it must become a VMEM load because caches are not coherent. That said you cannot reorder SMRD with aliasing VMEM store, both because of the coherency and because memory may be indeed overwritten.

This only happens for invariant MMOs. This isn't the only condition


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

https://reviews.llvm.org/D11621