[PATCH] D80530: [AMDGPU][MC][GFX908] Disabled SGPRs as src0 of v_accvgpr_write

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:3167-3170
+  if (isSGPR(mc2PseudoReg(Reg), TRI)) {
+    Error(getLoc(), "source operand must be either a VGPR or an inline constant");
+    return false;
+  }
----------------
This isn't checking the constant values as the error implies


================
Comment at: llvm/test/MC/AMDGPU/mai.s:42
+// NOGFX908: error: source operand must be either a VGPR or an inline constant
+
 v_mfma_f32_32x32x1f32 a[0:31], v0, v1, a[1:32]
----------------
Test with inline immediate and non-inline immediate constants?


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

https://reviews.llvm.org/D80530