[llvm] [TableGen] Remove unnecessary use of utostr when writing to raw_ostream. NFC (PR #154800)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 21 10:21:04 PDT 2025


https://github.com/topperc created https://github.com/llvm/llvm-project/pull/154800

raw_ostream is capable of printing unsigned or uint64_t directly.

>From 1939bd339e88ef5b3d2df31c73aaafefad038b1e Mon Sep 17 00:00:00 2001
From: Craig Topper <craig.topper at sifive.com>
Date: Thu, 21 Aug 2025 10:00:31 -0700
Subject: [PATCH] [TableGen] Remove unnecessary use of utostr when writing to
 raw_ostream. NFC

---
 llvm/utils/TableGen/CodeEmitterGen.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/llvm/utils/TableGen/CodeEmitterGen.cpp b/llvm/utils/TableGen/CodeEmitterGen.cpp
index d7b5e21c3f1fb..a58fa9f79a8f0 100644
--- a/llvm/utils/TableGen/CodeEmitterGen.cpp
+++ b/llvm/utils/TableGen/CodeEmitterGen.cpp
@@ -154,9 +154,9 @@ bool CodeEmitterGen::addCodeToMergeInOperand(const Record *R,
     raw_string_ostream CaseOS(Case);
     CaseOS << indent(6);
     if (UseAPInt) {
-      CaseOS << EncoderMethodName << "(MI, " + utostr(OpIdx) << ", op";
+      CaseOS << EncoderMethodName << "(MI, " << OpIdx << ", op";
     } else {
-      CaseOS << "op = " << EncoderMethodName << "(MI, " << utostr(OpIdx);
+      CaseOS << "op = " << EncoderMethodName << "(MI, " << OpIdx;
     }
     CaseOS << ", Fixups, STI);\n";
   } else {
@@ -388,8 +388,7 @@ void CodeEmitterGen::addInstructionCasesForEncoding(
 
 static void emitInstBits(raw_ostream &OS, const APInt &Bits) {
   for (unsigned I = 0; I < Bits.getNumWords(); ++I)
-    OS << ((I > 0) ? ", " : "") << "UINT64_C(" << utostr(Bits.getRawData()[I])
-       << ")";
+    OS << ((I > 0) ? ", " : "") << "UINT64_C(" << Bits.getRawData()[I] << ")";
 }
 
 void CodeEmitterGen::emitInstructionBaseValues(



More information about the llvm-commits mailing list