[PATCH] D79435: [AMDGPU] Drop 16 bit subreg suffixes on print

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 5 11:20:15 PDT 2020


rampitec marked an inline comment as done.
rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUInstPrinter.cpp:29-33
+static cl::opt<bool> Keep16BitSuffixes(
+  "amdgpu-keep-16-bit-reg-suffixes",
+  cl::desc("Keep .l and .h suffixes in asm for debugging purposes"),
+  cl::init(false),
+  cl::ReallyHidden);
----------------
arsenm wrote:
> Could this be implemented as an assembler variant (e.g. AT&T vs. intel syntax?)
Probably. I did not explore this.


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

https://reviews.llvm.org/D79435





More information about the llvm-commits mailing list