[llvm-commits] [llvm] r76789 - in /llvm/trunk: lib/CodeGen/ScheduleDAGPrinter.cpp lib/Target/PIC16/PIC16DebugInfo.cpp lib/Target/TargetAsmInfo.cpp lib/VMCore/Mangler.cpp tools/lto/LTOCodeGenerator.cpp tools/opt/PrintSCC.cpp

Daniel Dunbar daniel at zuster.org
Wed Jul 22 14:33:09 PDT 2009


Author: ddunbar
Date: Wed Jul 22 16:33:09 2009
New Revision: 76789

URL: http://llvm.org/viewvc/llvm-project?rev=76789&view=rev
Log:
Switch some clients to Value::getName(), and other getName() user
simplification.
 - NFC

Modified:
    llvm/trunk/lib/CodeGen/ScheduleDAGPrinter.cpp
    llvm/trunk/lib/Target/PIC16/PIC16DebugInfo.cpp
    llvm/trunk/lib/Target/TargetAsmInfo.cpp
    llvm/trunk/lib/VMCore/Mangler.cpp
    llvm/trunk/tools/lto/LTOCodeGenerator.cpp
    llvm/trunk/tools/opt/PrintSCC.cpp

Modified: llvm/trunk/lib/CodeGen/ScheduleDAGPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/ScheduleDAGPrinter.cpp?rev=76789&r1=76788&r2=76789&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/ScheduleDAGPrinter.cpp (original)
+++ llvm/trunk/lib/CodeGen/ScheduleDAGPrinter.cpp Wed Jul 22 16:33:09 2009
@@ -87,8 +87,8 @@
 #ifndef NDEBUG
   if (BB->getBasicBlock())
     ViewGraph(this, "dag." + MF.getFunction()->getName(), false,
-              "Scheduling-Units Graph for " + MF.getFunction()->getName() + ':' +
-              BB->getBasicBlock()->getName());
+              "Scheduling-Units Graph for " + MF.getFunction()->getName() + 
+              ":" + BB->getBasicBlock()->getName());
   else
     ViewGraph(this, "dag." + MF.getFunction()->getName(), false,
               "Scheduling-Units Graph for " + MF.getFunction()->getName());

Modified: llvm/trunk/lib/Target/PIC16/PIC16DebugInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PIC16/PIC16DebugInfo.cpp?rev=76789&r1=76788&r2=76789&view=diff

==============================================================================
--- llvm/trunk/lib/Target/PIC16/PIC16DebugInfo.cpp (original)
+++ llvm/trunk/lib/Target/PIC16/PIC16DebugInfo.cpp Wed Jul 22 16:33:09 2009
@@ -327,7 +327,7 @@
     // Structures and union declaration's debug info has llvm.dbg.composite
     // in its name.
     // FIXME: Checking and relying on llvm.dbg.composite name is not a good idea.
-    if(I->getName().find("llvm.dbg.composite") != std::string::npos) {
+    if(I->getNameStr().find("llvm.dbg.composite") != std::string::npos) {
       GlobalVariable *GV = cast<GlobalVariable >(I);
       DICompositeType CTy(GV);
       if (CTy.getTag() == dwarf::DW_TAG_union_type ||

Modified: llvm/trunk/lib/Target/TargetAsmInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/TargetAsmInfo.cpp?rev=76789&r1=76788&r2=76789&view=diff

==============================================================================
--- llvm/trunk/lib/Target/TargetAsmInfo.cpp (original)
+++ llvm/trunk/lib/Target/TargetAsmInfo.cpp Wed Jul 22 16:33:09 2009
@@ -349,33 +349,33 @@
                                       SectionKind::Kind Kind) const {
   switch (Kind) {
   case SectionKind::Text:
-    return ".gnu.linkonce.t." + GV->getName();
+    return ".gnu.linkonce.t." + GV->getNameStr();
   case SectionKind::Data:
-    return ".gnu.linkonce.d." + GV->getName();
+    return ".gnu.linkonce.d." + GV->getNameStr();
   case SectionKind::DataRel:
-    return ".gnu.linkonce.d.rel" + GV->getName();
+    return ".gnu.linkonce.d.rel" + GV->getNameStr();
   case SectionKind::DataRelLocal:
-    return ".gnu.linkonce.d.rel.local" + GV->getName();
+    return ".gnu.linkonce.d.rel.local" + GV->getNameStr();
   case SectionKind::DataRelRO:
-    return ".gnu.linkonce.d.rel.ro" + GV->getName();
+    return ".gnu.linkonce.d.rel.ro" + GV->getNameStr();
   case SectionKind::DataRelROLocal:
-    return ".gnu.linkonce.d.rel.ro.local" + GV->getName();
+    return ".gnu.linkonce.d.rel.ro.local" + GV->getNameStr();
   case SectionKind::SmallData:
-    return ".gnu.linkonce.s." + GV->getName();
+    return ".gnu.linkonce.s." + GV->getNameStr();
   case SectionKind::BSS:
-    return ".gnu.linkonce.b." + GV->getName();
+    return ".gnu.linkonce.b." + GV->getNameStr();
   case SectionKind::SmallBSS:
-    return ".gnu.linkonce.sb." + GV->getName();
+    return ".gnu.linkonce.sb." + GV->getNameStr();
   case SectionKind::ROData:
   case SectionKind::RODataMergeConst:
   case SectionKind::RODataMergeStr:
-    return ".gnu.linkonce.r." + GV->getName();
+    return ".gnu.linkonce.r." + GV->getNameStr();
   case SectionKind::SmallROData:
-    return ".gnu.linkonce.s2." + GV->getName();
+    return ".gnu.linkonce.s2." + GV->getNameStr();
   case SectionKind::ThreadData:
-    return ".gnu.linkonce.td." + GV->getName();
+    return ".gnu.linkonce.td." + GV->getNameStr();
   case SectionKind::ThreadBSS:
-    return ".gnu.linkonce.tb." + GV->getName();
+    return ".gnu.linkonce.tb." + GV->getNameStr();
   default:
     llvm_unreachable("Unknown section kind");
   }

Modified: llvm/trunk/lib/VMCore/Mangler.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/Mangler.cpp?rev=76789&r1=76788&r2=76789&view=diff

==============================================================================
--- llvm/trunk/lib/VMCore/Mangler.cpp (original)
+++ llvm/trunk/lib/VMCore/Mangler.cpp Wed Jul 22 16:33:09 2009
@@ -146,7 +146,7 @@
       GV->hasLinkerPrivateLinkage() ? Mangler::LinkerPrivate : Mangler::Default;
 
   if (GV->hasName())
-    return makeNameProper(GV->getName() + Suffix, PrefixTy);
+    return makeNameProper(GV->getNameStr() + Suffix, PrefixTy);
   
   // Get the ID for the global, assigning a new one if we haven't got one
   // already.

Modified: llvm/trunk/tools/lto/LTOCodeGenerator.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/LTOCodeGenerator.cpp?rev=76789&r1=76788&r2=76789&view=diff

==============================================================================
--- llvm/trunk/tools/lto/LTOCodeGenerator.cpp (original)
+++ llvm/trunk/tools/lto/LTOCodeGenerator.cpp Wed Jul 22 16:33:09 2009
@@ -375,13 +375,13 @@
                                         e = mergedModule->end(); f != e; ++f) {
                 if ( !f->isDeclaration() 
                   && _mustPreserveSymbols.count(mangler.getMangledName(f)) )
-                    mustPreserveList.push_back(::strdup(f->getName().c_str()));
+                  mustPreserveList.push_back(::strdup(f->getNameStr().c_str()));
             }
             for (Module::global_iterator v = mergedModule->global_begin(), 
                                  e = mergedModule->global_end(); v !=  e; ++v) {
                 if ( !v->isDeclaration()
                   && _mustPreserveSymbols.count(mangler.getMangledName(v)) )
-                    mustPreserveList.push_back(::strdup(v->getName().c_str()));
+                  mustPreserveList.push_back(::strdup(v->getNameStr().c_str()));
             }
             passes.add(createInternalizePass(mustPreserveList));
         }

Modified: llvm/trunk/tools/opt/PrintSCC.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/opt/PrintSCC.cpp?rev=76789&r1=76788&r2=76789&view=diff

==============================================================================
--- llvm/trunk/tools/opt/PrintSCC.cpp (original)
+++ llvm/trunk/tools/opt/PrintSCC.cpp Wed Jul 22 16:33:09 2009
@@ -101,7 +101,7 @@
     outs() << "\nSCC #" << ++sccNum << " : ";
     for (std::vector<CallGraphNode*>::const_iterator I = nextSCC.begin(),
            E = nextSCC.end(); I != E; ++I)
-      outs() << ((*I)->getFunction() ? (*I)->getFunction()->getName()
+      outs() << ((*I)->getFunction() ? (*I)->getFunction()->getNameStr()
                  : std::string("Indirect CallGraph node")) << ", ";
     if (nextSCC.size() == 1 && SCCI.hasLoop())
       outs() << " (Has self-loop).";





More information about the llvm-commits mailing list