[clang] Deduplicate the logic between StringMapEntry.first() and StringMapEntry.getKey().

bricknerb · bricknerb · commit cf1ff225026c · 2024-10-25T21:55:25.000+02:00
diff --git a/llvm/include/llvm/ADT/StringMapEntry.h b/llvm/include/llvm/ADT/StringMapEntry.h
@@ -117,7 +117,7 @@ class StringMapEntry final : public StringMapEntryStorage<ValueTy> {
   }
 
   StringRef first() const {
-    return StringRef(getKeyData(), this->getKeyLength());
+    return getKey();
   }
 
   /// Create a StringMapEntry for the specified key construct the value using

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ class StringMapEntry final : public StringMapEntryStorage<ValueTy> {`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`StringRef first() const {`
`120`		`- return StringRef(getKeyData(), this->getKeyLength());`
	`120`	`+ return getKey();`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`/// Create a StringMapEntry for the specified key construct the value using`