[clang] [clang][builtins] Return a `SmallString` from `Builtin::Context::getQuotedName()` (PR #227963)

Timm Baeder via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 3 22:40:17 PDT 2026


================
@@ -96,11 +96,13 @@ std::string Builtin::Context::getName(unsigned ID) const {
   return I.getName(Shard);
 }
 
-std::string Builtin::Context::getQuotedName(unsigned ID) const {
+SmallString<64> Builtin::Context::getQuotedName(unsigned ID) const {
   const auto &[Shard, I] = getShardAndInfo(ID);
-  return (Twine("'") + Shard.NamePrefix + (*Shard.Strings)[I.Offsets.Name] +
-          "'")
-      .str();
+  SmallString<64> Result("'");
+  Result += Shard.NamePrefix;
+  Result += (*Shard.Strings)[I.Offsets.Name];
+  Result += '\'';
----------------
tbaederr wrote:

Tried the reserve here but that didn't work out: https://llvm-compile-time-tracker.com/compare.php?from=911f63c8ac9486fe5b65f80f4cadd0db4abe7cf3&to=87da13500f7540200213b1900fdd17f926e1159b&stat=instructions:u

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


More information about the cfe-commits mailing list