[PATCH] D64892: [AMDGPU] Simplify AMDGPUInstPrinter::printRegOperand()

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/MCTargetDesc/AMDGPUInstPrinter.cpp:323
+
+  O << StringRef(getRegisterName(RegNo, AltName)).lower();
 }
----------------
rampitec wrote:
> arsenm wrote:
> > Is the lower() still necessary?
> I can lowercase all registers in td, then it will be not necessary.
I thought the new generated names were lowercase already


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

https://reviews.llvm.org/D64892