[PATCH] D115229: [AMDGPU] In SIMemoryLegalizer assume all atomic ops have memoperands

[Tony Tye via Phabricator via llvm-commits]

t-tye added a comment.

My understanding was that it was not guaranteed that an MI instruction has a MMO. That passes could drop the MMO. Are you saying that is not the case? As Matt says, if that is the case the verifier needs to enforce that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115229