From 2bbb7e3ca609dcd2eb2ae3b98d85fe002baf653e Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Tue, 15 Nov 2011 18:30:12 +0000 Subject: [PATCH] Remove Value::getNameStr. It has been deprecated for a while and provides no additional value over getName(). git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@144657 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Value.h | 11 ----------- lib/VMCore/Value.cpp | 4 ---- 2 files changed, 15 deletions(-) diff --git a/include/llvm/Value.h b/include/llvm/Value.h index a71e2fdefd7..9b47143b7fd 100644 --- a/include/llvm/Value.h +++ b/include/llvm/Value.h @@ -117,19 +117,8 @@ public: /// getName() - Return a constant reference to the value's name. This is cheap /// and guaranteed to return the same reference as long as the value is not /// modified. - /// - /// This is currently guaranteed to return a StringRef for which data() points - /// to a valid null terminated string. The use of StringRef.data() is - /// deprecated here, however, and clients should not rely on it. If such - /// behavior is needed, clients should use expensive getNameStr(), or switch - /// to an interface that does not depend on null termination. StringRef getName() const; - /// getNameStr() - Return the name of the specified value, *constructing a - /// string* to hold it. This is guaranteed to construct a string and is very - /// expensive, clients should use getName() unless necessary. - std::string getNameStr() const; - /// setName() - Change the name of the value, choosing a new unique name if /// the provided name is taken. /// diff --git a/lib/VMCore/Value.cpp b/lib/VMCore/Value.cpp index b9a1e54935d..291df917706 100644 --- a/lib/VMCore/Value.cpp +++ b/lib/VMCore/Value.cpp @@ -156,10 +156,6 @@ StringRef Value::getName() const { return Name->getKey(); } -std::string Value::getNameStr() const { - return getName().str(); -} - void Value::setName(const Twine &NewName) { // Fast path for common IRBuilder case of setName("") when there is no name. if (NewName.isTriviallyEmpty() && !hasName())