[PATCH] D127314: [AMDGPU][GFX9][GFX10] Support base+soffset+offset SMEM atomics.

[Ivan Kosarev via Phabricator via llvm-commits]

kosarev added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SMInstructions.td:90
 
+class OffsetMode<bit hasOffset, bit hasSOffset, string variant,
+                 dag ins, string asm> {
----------------
dp wrote:
> Nice idea! Could other classes for loads, stores, etc. benefit from using `OffsetMode`?
That's the plan, yes, but one step at a time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127314