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

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

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.


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

https://reviews.llvm.org/D11621