[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