Fix up a few minor performance problems spotted in code review.
authorEric Christopher <echristo@gmail.com>
Sat, 19 Oct 2013 01:04:42 +0000 (01:04 +0000)
committerEric Christopher <echristo@gmail.com>
Sat, 19 Oct 2013 01:04:42 +0000 (01:04 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@193023 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp

index c39320e149fefa0189cf7c11ef6fe71e82b6e06c..30cd27bae003540d1851ce2a4fab9dbdde5f3c6e 100644 (file)
@@ -912,8 +912,7 @@ void CompileUnit::addAccelType(StringRef Name, std::pair<DIE *, unsigned> Die) {
 
 /// addGlobalName - Add a new global name to the compile unit.
 void CompileUnit::addGlobalName(StringRef Name, DIE *Die, DIScope Context) {
-  std::string ContextString = getParentContextString(Context);
-  std::string FullName = ContextString + Name.str();
+  std::string FullName = getParentContextString(Context) + Name.str();
   GlobalNames[FullName] = Die;
 }
 
@@ -925,9 +924,9 @@ void CompileUnit::addGlobalType(DIType Ty) {
       (!Context || Context.isCompileUnit() || Context.isFile() ||
        Context.isNameSpace()))
     if (DIEEntry *Entry = getDIEEntry(Ty)) {
-      std::string ContextString = getParentContextString(Context);
-      std::string FullName = ContextString + Ty.getName().str();
-      GlobalTypes[FullName] = Entry->getEntry();
+      std::string FullName =
+          getParentContextString(Context) + Ty.getName().str();
+       GlobalTypes[FullName] = Entry->getEntry();
     }
 }
 
@@ -944,7 +943,7 @@ std::string CompileUnit::getParentContextString(DIScope Context) const {
   if (getLanguage() != dwarf::DW_LANG_C_plus_plus)
     return "";
 
-  std::string CS = "";
+  std::string CS;
   SmallVector<DIScope, 1> Parents;
   while (!Context.isCompileUnit()) {
     Parents.push_back(Context);
@@ -963,7 +962,7 @@ std::string CompileUnit::getParentContextString(DIScope Context) const {
        I != E; ++I) {
     DIScope Ctx = *I;
     StringRef Name = Ctx.getName();
-    if (Name != "") {
+    if (!Name.empty()) {
       CS += Name;
       CS += "::";
     }