[PATCH] D152311: [RISCV] Merge RISCVMCInstLower.cpp into RISCVAsmPrinter.cpp.

Sergei Barannikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 9 10:04:03 PDT 2023


barannikov88 accepted this revision.
barannikov88 added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/RISCV/RISCVAsmPrinter.cpp:159
 
   MCInst TmpInst;
+  if (!lowerToMCInst(MI, TmpInst))
----------------
(nit) Can this be renamed to OutMI or OutInst or something more meaningful?



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