[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