[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