[PATCH] Remove unused method in dsymutil DwarfLinker

Xan López xan at igalia.com
Thu Jul 16 09:36:27 PDT 2015


SSIA.

(I do not have commit rights, please push the patch for me if it's accepted)
-------------- next part --------------
>From 83c2ffb40db4baf9892ea0510e1c770bef7f0949 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Xan=20L=C3=B3pez?= <xan at igalia.com>
Date: Thu, 16 Jul 2015 18:29:45 +0200
Subject: [PATCH] DwarkLinker: remove unused method

---
 tools/dsymutil/DwarfLinker.cpp | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/tools/dsymutil/DwarfLinker.cpp b/tools/dsymutil/DwarfLinker.cpp
index 6e9087c..86c22dd 100644
--- a/tools/dsymutil/DwarfLinker.cpp
+++ b/tools/dsymutil/DwarfLinker.cpp
@@ -324,12 +324,6 @@ public:
   /// one.
   uint32_t getStringOffset(StringRef S);
 
-  /// \brief Get permanent storage for \p S (but do not necessarily
-  /// emit \p S in the output section).
-  /// \returns The StringRef that points to permanent storage to use
-  /// in place of \p S.
-  StringRef internString(StringRef S);
-
   // \brief Return the first entry of the string table.
   const MapTy::MapEntryTy *getFirstEntry() const {
     return getNextEntry(&Sentinel);
@@ -373,16 +367,6 @@ uint32_t NonRelocatableStringpool::getStringOffset(StringRef S) {
   return It->getValue().first;
 }
 
-/// \brief Put \p S into the StringMap so that it gets permanent
-/// storage, but do not actually link it in the chain of elements
-/// that go into the output section. A latter call to
-/// getStringOffset() with the same string will chain it though.
-StringRef NonRelocatableStringpool::internString(StringRef S) {
-  std::pair<uint32_t, StringMapEntryBase *> Entry(0, nullptr);
-  auto InsertResult = Strings.insert(std::make_pair(S, Entry));
-  return InsertResult.first->getKey();
-}
-
 /// \brief The Dwarf streaming logic
 ///
 /// All interactions with the MC layer that is used to build the debug
-- 
2.4.3



More information about the llvm-commits mailing list