[PATCH] D24623: AMDGPU: Implement memory model

[Tony Tye via Phabricator via llvm-commits]

t-tye accepted this revision.
t-tye added a comment.
This revision is now accepted and ready to land.

One question, otherwise LGTM.



================
Comment at: lib/Target/AMDGPU/SIInstructions.td:102
   let hasSideEffects = 1;
+  let maybeAtomic = 1;
 }
----------------
Since a fence itself does not modify memory, and has no memory address operand, should it be marked `maybeatomic`?


https://reviews.llvm.org/D24623