[PATCH] D105507: [AMDGPU] Add gfx10 assembler directive to specify shared VGPR count

Jacob Lambert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 31 19:08:36 PST 2022


lamb-j added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:4880
+    // SharedVGPRCount < 16 checked by PARSE_ENTRY_BITS
+    if (SharedVGPRCount && EnableWavefrontSize32)
+      return TokError("shared_vgpr_count directive not valid on "
----------------
arsenm wrote:
> Braces
Do we want braces here because the TokError() string is split over two lines?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105507



More information about the llvm-commits mailing list