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

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 19 05:35:44 PDT 2019


asb requested changes to this revision.
asb added a comment.
This revision now requires changes to proceed.

Actually changing my mind. This would be better tested by adopting something like arch-reg-names.s from D66139 <https://reviews.llvm.org/D66139> (and extending that to handle FP regs), and leaving the inline-asm tests alone (huge increase in test size, which makes them harder to follow). Make that change, and then it looks good to me.


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