[PATCH] D125700: [AMDGPU][GFX9] Support base+soffset+offset SMEM loads.

[Dmitry Preobrazhensky via Phabricator via llvm-commits]

dp added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SMInstructions.td:570
+    !if(ps.has_soffset, soffset{6-0}, ?),
+    !if(!and(ps.has_offset, ps.has_soffset), soffset{6-0}, ?));
 }
----------------
Now that we have IsGFX9Specific, could not this expression be replaced with '?'


================
Comment at: llvm/lib/Target/AMDGPU/SMInstructions.td:587
+    let SOffsetEn = 1, Offset = ?, Subtarget = SIEncodingFamily.GFX9,
+        AsmVariantName = "NonParsable" in
+    def _SGPR_alt_gfx9 : SMEM_Real_Load_vi <op, ps#"_SGPR",
----------------
Ingenious!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125700