[lld] r276253 - [ELF] - Linkerscript: remove excessive ScriptConfiguration::Filler field
George Rimar via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 21 00:48:55 PDT 2016
Author: grimar
Date: Thu Jul 21 02:48:54 2016
New Revision: 276253
URL: http://llvm.org/viewvc/llvm-project?rev=276253&view=rev
Log:
[ELF] - Linkerscript: remove excessive ScriptConfiguration::Filler field
Previously OutputSectionCommand::Filler was introduced, but unused.
Patch fixes that.
Differential revision: https://reviews.llvm.org/D22615
Modified:
lld/trunk/ELF/LinkerScript.cpp
lld/trunk/ELF/LinkerScript.h
Modified: lld/trunk/ELF/LinkerScript.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/LinkerScript.cpp?rev=276253&r1=276252&r2=276253&view=diff
==============================================================================
--- lld/trunk/ELF/LinkerScript.cpp (original)
+++ lld/trunk/ELF/LinkerScript.cpp Thu Jul 21 02:48:54 2016
@@ -410,10 +410,11 @@ LinkerScript<ELFT>::createPhdrs(ArrayRef
template <class ELFT>
ArrayRef<uint8_t> LinkerScript<ELFT>::getFiller(StringRef Name) {
- auto I = Opt.Filler.find(Name);
- if (I == Opt.Filler.end())
- return {};
- return I->second;
+ for (const std::unique_ptr<BaseCommand> &Base : Opt.Commands)
+ if (auto *Cmd = dyn_cast<OutputSectionCommand>(Base.get()))
+ if (Cmd->Name == Name)
+ return Cmd->Filler;
+ return {};
}
// Returns the index of the given section name in linker script
@@ -754,7 +755,7 @@ void ScriptParser::readOutputSectionDesc
return;
}
Tok = Tok.substr(3);
- Opt.Filler[OutSec] = parseHex(Tok);
+ Cmd->Filler = parseHex(Tok);
next();
}
}
Modified: lld/trunk/ELF/LinkerScript.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/LinkerScript.h?rev=276253&r1=276252&r2=276253&view=diff
==============================================================================
--- lld/trunk/ELF/LinkerScript.h (original)
+++ lld/trunk/ELF/LinkerScript.h Thu Jul 21 02:48:54 2016
@@ -86,9 +86,6 @@ struct ScriptConfiguration {
// SECTIONS commands.
std::vector<SectionRule> Sections;
- // Section fill attribute for each section.
- llvm::StringMap<std::vector<uint8_t>> Filler;
-
// Used to assign addresses to sections.
std::vector<std::unique_ptr<BaseCommand>> Commands;
More information about the llvm-commits
mailing list