[PATCH] D152311: [RISCV] Merge RISCVMCInstLower.cpp into RISCVAsmPrinter.cpp.
Fraser Cormack via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 6 23:36:09 PDT 2023
frasercrmck added a comment.
I'm in favour of this. I think you explained the inconsistencies well. I believe RISCV has served as an example for new backends (I don't know if that's still the case) so I think there's a good argument (given the aforementioned inconsistencies) to forge a path forward.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152311/new/
https://reviews.llvm.org/D152311
More information about the llvm-commits
mailing list