From 0ab6439c3f0eef28b4b40f31f2d4d9732fe8cd15 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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@193023 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp b/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp index c39320e149f..30cd27bae00 100644 --- a/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp +++ b/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 += "::"; }