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

[bricknerb]

Zero behavior diff.

[llvmbot]

@llvm/pr-subscribers-llvm-adt

Author: Boaz Brickner (bricknerb)

Changes

Zero behavior diff.

---

Full diff: https://github.com/llvm/llvm-project/pull/113735.diff

1 Files Affected:

• (modified) llvm/include/llvm/ADT/StringMapEntry.h (+1-1)

diff --git a/llvm/include/llvm/ADT/StringMapEntry.h b/llvm/include/llvm/ADT/StringMapEntry.h
index 98b51cc1aebd59..a3ff2415869be6 100644
--- 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

[github-actions]

✅ With the latest revision this PR passed the C/C++ code formatter.

[dwblaikie]

Sounds good - please add [NFC] to the title

[bricknerb]

Sounds good - please add [NFC] to the title

Done.