From: Mehdi Amini <mehdi.amini@apple.com>
Date: Sat, 1 Oct 2016 06:25:30 +0000 (+0000)
Subject: Use StringRef instead of raw pointer in TargetRegistry API (NFC)
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=876fe65d272ea1fb270e1e2dc8ccda00d4006051;p=llvm

Use StringRef instead of raw pointer in TargetRegistry API (NFC)

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@283017 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/include/llvm/Support/TargetRegistry.h b/include/llvm/Support/TargetRegistry.h
index 99d679a31c0..9966ec97d78 100644
--- a/include/llvm/Support/TargetRegistry.h
+++ b/include/llvm/Support/TargetRegistry.h
@@ -165,10 +165,10 @@ private:
   ArchMatchFnTy ArchMatchFn;
 
   /// Name - The target name.
-  const char *Name;
+  StringRef Name;
 
   /// ShortDesc - A short description of the target.
-  const char *ShortDesc;
+  StringRef ShortDesc;
 
   /// HasJIT - Whether this target supports the JIT.
   bool HasJIT;
@@ -262,10 +262,10 @@ public:
   const Target *getNext() const { return Next; }
 
   /// getName - Get the target name.
-  const char *getName() const { return Name; }
+  StringRef getName() const { return Name; }
 
   /// getShortDescription - Get a short description of the target.
-  const char *getShortDescription() const { return ShortDesc; }
+  StringRef getShortDescription() const { return ShortDesc; }
 
   /// @}
   /// @name Feature Predicates
@@ -624,7 +624,7 @@ struct TargetRegistry {
   /// @param ArchMatchFn - The arch match checking function for this target.
   /// @param HasJIT - Whether the target supports JIT code
   /// generation.
-  static void RegisterTarget(Target &T, const char *Name, const char *ShortDesc,
+  static void RegisterTarget(Target &T, StringRef Name, StringRef ShortDesc,
                              Target::ArchMatchFnTy ArchMatchFn,
                              bool HasJIT = false);
 
@@ -857,7 +857,7 @@ struct TargetRegistry {
 template <Triple::ArchType TargetArchType = Triple::UnknownArch,
           bool HasJIT = false>
 struct RegisterTarget {
-  RegisterTarget(Target &T, const char *Name, const char *Desc) {
+  RegisterTarget(Target &T, StringRef Name, StringRef Desc) {
     TargetRegistry::RegisterTarget(T, Name, Desc, &getArchMatch, HasJIT);
   }
 
diff --git a/lib/Support/TargetRegistry.cpp b/lib/Support/TargetRegistry.cpp
index bed9ed64f80..1b8ad3a0103 100644
--- a/lib/Support/TargetRegistry.cpp
+++ b/lib/Support/TargetRegistry.cpp
@@ -78,8 +78,8 @@ const Target *TargetRegistry::lookupTarget(const std::string &TT,
 
   auto J = std::find_if(std::next(I), targets().end(), ArchMatch);
   if (J != targets().end()) {
-    Error = std::string("Cannot choose between targets \"") + I->Name +
-            "\" and \"" + J->Name + "\"";
+    Error = ("Cannot choose between targets \"" + I->Name +
+            "\" a nd \"" + J->Name + "\"").str();
     return nullptr;
   }
 
@@ -87,16 +87,16 @@ const Target *TargetRegistry::lookupTarget(const std::string &TT,
 }
 
 void TargetRegistry::RegisterTarget(Target &T,
-                                    const char *Name,
-                                    const char *ShortDesc,
+                                    StringRef Name,
+                                    StringRef ShortDesc,
                                     Target::ArchMatchFnTy ArchMatchFn,
                                     bool HasJIT) {
-  assert(Name && ShortDesc && ArchMatchFn &&
+  assert(!Name.empty() && !ShortDesc.empty() && ArchMatchFn &&
          "Missing required target information!");
 
   // Check if this target has already been initialized, we allow this as a
   // convenience to some clients.
-  if (T.Name)
+  if (!T.Name.empty())
     return;
          
   // Add to the list of targets.
diff --git a/lib/Target/TargetMachineC.cpp b/lib/Target/TargetMachineC.cpp
index 5fb5b022780..b20012fba32 100644
--- a/lib/Target/TargetMachineC.cpp
+++ b/lib/Target/TargetMachineC.cpp
@@ -81,11 +81,11 @@ LLVMBool LLVMGetTargetFromTriple(const char* TripleStr, LLVMTargetRef *T,
 }
 
 const char * LLVMGetTargetName(LLVMTargetRef T) {
-  return unwrap(T)->getName();
+  return unwrap(T)->getName().data();
 }
 
 const char * LLVMGetTargetDescription(LLVMTargetRef T) {
-  return unwrap(T)->getShortDescription();
+  return unwrap(T)->getShortDescription().data();
 }
 
 LLVMBool LLVMTargetHasJIT(LLVMTargetRef T) {
diff --git a/tools/llc/llc.cpp b/tools/llc/llc.cpp
index 1a6ff63b118..2c64cf33f59 100644
--- a/tools/llc/llc.cpp
+++ b/tools/llc/llc.cpp
@@ -159,7 +159,7 @@ static cl::opt<RunPassOption, true, cl::parser<std::string>> RunPass(
 static int compileModule(char **, LLVMContext &);
 
 static std::unique_ptr<tool_output_file>
-GetOutputStream(const char *TargetName, Triple::OSType OS,
+GetOutputStream(StringRef TargetName, Triple::OSType OS,
                 const char *ProgName) {
   // If we don't yet have an output filename, make one.
   if (OutputFilename.empty()) {