[PATCH] D65950: [RISCV] Add Option for Printing Architectural Register Names

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 8 07:01:59 PDT 2019


asb added inline comments.


================
Comment at: llvm/lib/Target/RISCV/MCTargetDesc/RISCVInstPrinter.cpp:44
+    NoABINames("riscv-no-abi-regs",
+               cl::desc("Print register names as x0-31 rather than the ABI name"),
+               cl::init(false), cl::Hidden);
----------------
The description makes it unclear that this affects FPRs as well. Maybe "architectural register names such as x0-x31 rather than the ABI name"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65950





More information about the llvm-commits mailing list