From: David Blaikie Date: Fri, 11 Apr 2014 22:49:14 +0000 (+0000) Subject: Pull out a named variable for the cached section names to aid readability. X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=06e433a099794743a63f459a89fe1d4321d5fcb5;p=oota-llvm.git Pull out a named variable for the cached section names to aid readability. Based on a code review suggestion from Eric Christopher in r205990 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@206080 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/MC/MCContext.cpp b/lib/MC/MCContext.cpp index 36e6b565adb..4152d6344e9 100644 --- a/lib/MC/MCContext.cpp +++ b/lib/MC/MCContext.cpp @@ -241,7 +241,8 @@ void MCContext::renameELFSection(const MCSectionELF *Section, StringRef Name) { auto I = ELFUniquingMap.insert(std::make_pair(SectionGroupPair(Name, GroupName), Section)).first; - const_cast(Section)->setSectionName(I->first.first); + StringRef CachedName = I->first.first; + const_cast(Section)->setSectionName(CachedName); } const MCSectionELF *MCContext:: @@ -262,8 +263,9 @@ getELFSection(StringRef Section, unsigned Type, unsigned Flags, if (!Group.empty()) GroupSym = GetOrCreateSymbol(Group); - MCSectionELF *Result = new (*this) MCSectionELF( - Entry.first.first, Type, Flags, Kind, EntrySize, GroupSym); + StringRef CachedName = Entry.first.first; + MCSectionELF *Result = new (*this) + MCSectionELF(CachedName, Type, Flags, Kind, EntrySize, GroupSym); Entry.second = Result; return Result; } @@ -291,9 +293,9 @@ MCContext::getCOFFSection(StringRef Section, unsigned Characteristics, if (!COMDATSymName.empty()) COMDATSymbol = GetOrCreateSymbol(COMDATSymName); - MCSectionCOFF *Result = - new (*this) MCSectionCOFF(Iter->first.first, Characteristics, - COMDATSymbol, Selection, Assoc, Kind); + StringRef CachedName = Iter->first.first; + MCSectionCOFF *Result = new (*this) MCSectionCOFF( + CachedName, Characteristics, COMDATSymbol, Selection, Assoc, Kind); Iter->second = Result; return Result;