[PATCH] D39912: AMDGPU/SI: Implement d16 support for image intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:4134
   {"lwe",     AMDGPUOperand::ImmTyLWE,   true, nullptr},
+  {"d16",     AMDGPUOperand::ImmTyD16,   true, nullptr},
   {"dmask",   AMDGPUOperand::ImmTyDMask, false, nullptr},
----------------
It's part of the asm string, so I don't think you need this?


================
Comment at: lib/Target/AMDGPU/InstPrinter/AMDGPUInstPrinter.cpp:220-223
+void AMDGPUInstPrinter::printD16(const MCInst *MI, unsigned OpNo,
+                                 const MCSubtargetInfo &STI, raw_ostream &O) {
+  printNamedBit(MI, OpNo, O, "d16");
+}
----------------
You shouldn't need this? The d16 string is hardcoded into the asm string?


https://reviews.llvm.org/D39912