[llvm] [LLVM][TableGen] Use StringRef for various members `CGIOperandList::OperandInfo` (PR #140625)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Mon May 19 16:20:12 PDT 2025


================
@@ -142,8 +142,8 @@ bool CodeEmitterGen::addCodeToMergeInOperand(const Record *R,
   }
 
   std::pair<unsigned, unsigned> SO = CGI.Operands.getSubOperandNumber(OpIdx);
-  std::string &EncoderMethodName =
-      CGI.Operands[SO.first].EncoderMethodNames[SO.second];
+  const std::string EncoderMethodName =
----------------
jurahul wrote:

I mean there's that extra line to define a raw_string_ostream. Its a nitpicky thing. Here's my other PR that streamline the code without needing to convert to string: https://github.com/llvm/llvm-project/pull/140644/files

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


More information about the llvm-commits mailing list