[PATCH] D142193: [AMDGPU] Remove unused immediate operand definitions.

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 20 04:12:45 PST 2023


kosarev added a comment.

> Shouldn't AMDGPUInstPrinter::printU8ImmOperand (and perhaps AMDGPUInstPrinter::printU32ImmOperand) be removed as well?

Of course! `printU32ImmOperand()` helps other functions and has to stay. Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142193



More information about the llvm-commits mailing list