From 8dba0d5ae958d385ac0c794c734cc52bdaccf27e Mon Sep 17 00:00:00 2001 From: Eric Christopher Date: Sat, 19 Oct 2013 01:04:42 +0000 Subject: [PATCH] Fix up a few minor performance problems spotted in code review. llvm-svn: 193023 --- llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp b/llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp index c39320e149fe..30cd27bae003 100644 --- a/llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp +++ b/llvm/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp @@ -912,8 +912,7 @@ void CompileUnit::addAccelType(StringRef Name, std::pair 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 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 += "::"; }