[lld] r280252 - [ELF] Linkerscript: define symbols outside SECTIONS
Petr Hosek via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 31 08:31:17 PDT 2016
Author: phosek
Date: Wed Aug 31 10:31:17 2016
New Revision: 280252
URL: http://llvm.org/viewvc/llvm-project?rev=280252&view=rev
Log:
[ELF] Linkerscript: define symbols outside SECTIONS
Symbol assignments outside of SECTIONS command need to be created
even when SECTIONS command is not used.
Differential Revision: https://reviews.llvm.org/D23751
Modified:
lld/trunk/ELF/LinkerScript.cpp
lld/trunk/ELF/LinkerScript.h
lld/trunk/ELF/Writer.cpp
lld/trunk/test/ELF/linkerscript/linkerscript-symbols.s
Modified: lld/trunk/ELF/LinkerScript.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/LinkerScript.cpp?rev=280252&r1=280251&r2=280252&view=diff
==============================================================================
--- lld/trunk/ELF/LinkerScript.cpp (original)
+++ lld/trunk/ELF/LinkerScript.cpp Wed Aug 31 10:31:17 2016
@@ -259,6 +259,16 @@ LinkerScript<ELFT>::createInputSectionLi
}
template <class ELFT>
+void LinkerScript<ELFT>::createAssignments() {
+ for (const std::unique_ptr<SymbolAssignment> &Cmd : Opt.Assignments) {
+ if (shouldDefine<ELFT>(Cmd.get()))
+ addRegular<ELFT>(Cmd.get());
+ if (Cmd->Sym)
+ cast<DefinedRegular<ELFT>>(Cmd->Sym)->Value = Cmd->Expression(0);
+ }
+}
+
+template <class ELFT>
void LinkerScript<ELFT>::createSections(OutputSectionFactory<ELFT> &Factory) {
for (const std::unique_ptr<BaseCommand> &Base1 : Opt.Commands) {
if (auto *Cmd = dyn_cast<SymbolAssignment>(Base1.get())) {
@@ -714,12 +724,16 @@ void ScriptParser::readVersionScript() {
void ScriptParser::readLinkerScript() {
while (!atEOF()) {
StringRef Tok = next();
- if (Handler Fn = Cmd.lookup(Tok))
+ if (Handler Fn = Cmd.lookup(Tok)) {
(this->*Fn)();
- else if (SymbolAssignment *Cmd = readProvideOrAssignment(Tok))
- Opt.Commands.emplace_back(Cmd);
- else
+ } else if (SymbolAssignment *Cmd = readProvideOrAssignment(Tok)) {
+ if (Opt.HasContents)
+ Opt.Commands.emplace_back(Cmd);
+ else
+ Opt.Assignments.emplace_back(Cmd);
+ } else {
setError("unknown directive: " + Tok);
+ }
}
}
Modified: lld/trunk/ELF/LinkerScript.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/LinkerScript.h?rev=280252&r1=280251&r2=280252&view=diff
==============================================================================
--- lld/trunk/ELF/LinkerScript.h (original)
+++ lld/trunk/ELF/LinkerScript.h Wed Aug 31 10:31:17 2016
@@ -118,6 +118,8 @@ struct PhdrsCommand {
// ScriptConfiguration holds linker script parse results.
struct ScriptConfiguration {
+ // Used to create symbol assignments outside SECTIONS command.
+ std::vector<std::unique_ptr<SymbolAssignment>> Assignments;
// Used to assign addresses to sections.
std::vector<std::unique_ptr<BaseCommand>> Commands;
@@ -142,6 +144,7 @@ template <class ELFT> class LinkerScript
public:
LinkerScript();
~LinkerScript();
+ void createAssignments();
void createSections(OutputSectionFactory<ELFT> &Factory);
std::vector<PhdrEntry<ELFT>> createPhdrs();
Modified: lld/trunk/ELF/Writer.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Writer.cpp?rev=280252&r1=280251&r2=280252&view=diff
==============================================================================
--- lld/trunk/ELF/Writer.cpp (original)
+++ lld/trunk/ELF/Writer.cpp Wed Aug 31 10:31:17 2016
@@ -249,6 +249,8 @@ template <class ELFT> void Writer<ELFT>:
CommonInputSection<ELFT> Common(getCommonSymbols<ELFT>());
CommonInputSection<ELFT>::X = &Common;
+ Script<ELFT>::X->createAssignments();
+
Script<ELFT>::X->OutputSections = &OutputSections;
if (ScriptConfig->HasContents)
Script<ELFT>::X->createSections(Factory);
Modified: lld/trunk/test/ELF/linkerscript/linkerscript-symbols.s
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/ELF/linkerscript/linkerscript-symbols.s?rev=280252&r1=280251&r2=280252&view=diff
==============================================================================
--- lld/trunk/test/ELF/linkerscript/linkerscript-symbols.s (original)
+++ lld/trunk/test/ELF/linkerscript/linkerscript-symbols.s Wed Aug 31 10:31:17 2016
@@ -67,6 +67,15 @@
# RUN: llvm-objdump -t %t1 | FileCheck --check-prefix=HIDDEN5 %s
# HIDDEN5: 0000000000000000 *ABS* 00000000 somesym
+# Simple symbol assignment. All three symbols should have the
+# same value.
+# RUN: echo "foo = 0x100; SECTIONS { bar = foo; } baz = bar;" > %t.script
+# RUN: ld.lld -o %t1 --script %t.script %t
+# RUN: llvm-objdump -t %t1 | FileCheck --check-prefix=SIMPLE2 %s
+# SIMPLE2: 0000000000000100 *ABS* 00000000 foo
+# SIMPLE2: 0000000000000100 *ABS* 00000000 bar
+# SIMPLE2: 0000000000000100 *ABS* 00000000 baz
+
.global _start
_start:
nop
More information about the llvm-commits
mailing list