[PATCH] D125700: [AMDGPU][GFX9] Support base+soffset+offset SMEM loads.
Dmitry Preobrazhensky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 25 09:11:44 PDT 2022
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
More information about the llvm-commits
mailing list