[PATCH] D98397: [AMDGPU] Restrict image_msaa_load to MSAA dimension types

Dmitry Preobrazhensky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 06:02:02 PST 2021


dp added a comment.

MC changes look fine.



================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:4156
+  if (!validateMIMGMSAA(Inst)) {
+    Error(IDLoc, "invalid dim; must be MSAA type");
+    return false;
----------------
You could improve error position using

```
    getImmLoc(AMDGPUOperand::ImmTyDim, Operands)
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98397



More information about the llvm-commits mailing list