my mistake, Mangler::makeNameProper wants to take a twine, not a stringref!
authorChris Lattner <sabre@nondot.org>
Wed, 13 Jan 2010 05:02:57 +0000 (05:02 +0000)
committerChris Lattner <sabre@nondot.org>
Wed, 13 Jan 2010 05:02:57 +0000 (05:02 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@93296 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Support/Mangler.h
lib/VMCore/Mangler.cpp

index 9b885d054970e73fb4279d8937c3ab505bb2fd09..3fe88c1126aecdd313e6591a1527c5a98e604455 100644 (file)
@@ -19,7 +19,7 @@
 #include <string>
 
 namespace llvm {
-class StringRef;
+class Twine;
 class Type;
 class Module;
 class Value;
@@ -112,7 +112,7 @@ public:
   /// does this for you, so there's no point calling it on the result
   /// from getValueName.
   ///
-  std::string makeNameProper(StringRef x,
+  std::string makeNameProper(const Twine &Name,
                              ManglerPrefixTy PrefixTy = Mangler::Default);
   
   /// getNameWithPrefix - Fill OutName with the name of the appropriate prefix
index 2f546b11e466b6b197668a18d02b55e8e9709fa3..5783ddfba94359bb1ee10b5d5a87381e03c9f6af 100644 (file)
@@ -16,7 +16,7 @@
 #include "llvm/ADT/STLExtras.h"
 #include "llvm/ADT/StringExtras.h"
 #include "llvm/ADT/StringMap.h"
-#include "llvm/ADT/SmallVector.h"
+#include "llvm/ADT/SmallString.h"
 #include "llvm/Support/raw_ostream.h"
 using namespace llvm;
 
@@ -32,8 +32,11 @@ static std::string MangleLetter(unsigned char C) {
 /// makeNameProper - We don't want identifier names non-C-identifier characters
 /// in them, so mangle them as appropriate.
 ///
-std::string Mangler::makeNameProper(StringRef X,
+std::string Mangler::makeNameProper(const Twine &TheName,
                                     ManglerPrefixTy PrefixTy) {
+  SmallString<256> TmpData;
+  TheName.toVector(TmpData);
+  StringRef X = TmpData.str();
   assert(!X.empty() && "Cannot mangle empty strings");
   
   if (!UseQuotes) {