[PATCH] D64892: [AMDGPU] Simplify AMDGPUInstPrinter::printRegOperand()
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 17 15:43:32 PDT 2019
rampitec marked an inline comment as done.
rampitec added inline comments.
================
Comment at: lib/Target/AMDGPU/MCTargetDesc/AMDGPUInstPrinter.cpp:323
+
+ O << StringRef(getRegisterName(RegNo, AltName)).lower();
}
----------------
arsenm wrote:
> Is the lower() still necessary?
I can lowercase all registers in td, then it will be not necessary.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64892/new/
https://reviews.llvm.org/D64892
More information about the llvm-commits
mailing list