[llvm] [TARGETS-PARSER] Added const reference for params with size >= 16 bytes (PR #125083)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 21:38:16 PST 2025


================
@@ -32,7 +32,7 @@ namespace adjust {
 using namespace llvm;
 
 static void unsigned_width(unsigned Width, uint64_t Value,
-                           std::string Description, const MCFixup &Fixup,
+                           const std::string &Description, const MCFixup &Fixup,
----------------
arsenm wrote:

This should be using StringRef, and the code below should be using Twine instead of building the string itself 

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


More information about the llvm-commits mailing list