[PATCH] D147241: [AMDGPU][AsmParser] Refine parsing SDWA operands.

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 30 10:56:38 PDT 2023


kosarev added a comment.

> Can you say a little bit more than "refine" (here and in other patches)? I can see you capitalised Sdwa and introduced a new SDWAOperand class but what's the bigger picture? I guess the win here is being able to remove some code from `AMDGPUAsmParser.cpp`?

Yes, same as with other similar patches this change removes the need for the custom code in `parseCustomOperand()`. Do you suggest the commit message to say that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147241



More information about the llvm-commits mailing list