[llvm] [NFC][LLVM] Use `StringRef` for Modifier in Inst/Asm Printers (PR #135403)

Sergei Barannikov via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 11 12:55:50 PDT 2025


================
@@ -31,7 +31,7 @@ class RISCVInstPrinter : public MCInstPrinter {
   void printRegName(raw_ostream &O, MCRegister Reg) override;
 
   void printOperand(const MCInst *MI, unsigned OpNo, const MCSubtargetInfo &STI,
-                    raw_ostream &O, const char *Modifier = nullptr);
+                    raw_ostream &O, StringRef Modifier = {});
----------------
s-barannikov wrote:

The parameter has been deprecated a long time ago. It should be removed from function declarations wherever possible. RISCV version is never called with the Modifier. I didn't check other targets, but I guess half of them don't need the parameter either.


https://github.com/llvm/llvm-project/pull/135403


More information about the llvm-commits mailing list