[PATCH] D125907: [AMDGPU][MC][GFX8+] Correct SMEM offset parsing

[Dmitry Preobrazhensky via Phabricator via llvm-commits]

dp added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:7733
 bool AMDGPUOperand::isSMEMOffset() const {
-  return isImm(); // Offset range is checked later by validator.
+  return isImm() && !isCPol(); // Offset range is checked later by validator.
 }
----------------
kosarev wrote:
> Can we just say something like `isImmTy(ImmTyNone) || isImmTy(ImmTyOffset)`? The way it is, it's not clear what's so special about cache policies in relation with SMEM offsets.
Good point, thanks!


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

https://reviews.llvm.org/D125907