[lld] [LLD] Implement --enable-non-contiguous-regions (PR #90007)
Daniel Thornburgh via llvm-commits
llvm-commits at lists.llvm.org
Fri May 10 16:32:12 PDT 2024
================
@@ -1364,6 +1436,117 @@ const Defined *LinkerScript::assignAddresses() {
return getChangedSymbolAssignment(oldValues);
}
+static bool hasRegionOverflowed(MemoryRegion *mr) {
+ if (!mr)
+ return false;
+ return mr->curPos - mr->getOrigin() > mr->getLength();
+}
+
+// Spill input sections in reverse order of address assignment to (potentially)
+// bring memory regions out of overflow. The size savings of a spill can only be
+// estimated, since general linker script arithmetic may occur afterwards.
+// Under-estimates may cause unnecessary spills, but over-estimates can always
+// be corrected on the next pass.
+bool LinkerScript::spillSections() {
+ if (!config->enableNonContiguousRegions)
+ return false;
+
+ bool spilled = false;
+ for (SectionCommand *cmd : reverse(sectionCommands)) {
+ auto *od = dyn_cast<OutputDesc>(cmd);
+ if (!od)
+ continue;
+ OutputSection *osec = &od->osec;
+ if (!osec->size || !osec->memRegion)
+ continue;
+
+ // Input sections that have replaced a potential spill and should be removed
+ // from their input section description.
+ DenseSet<InputSection *> spilledInputSections;
+
+ for (SectionCommand *cmd : reverse(osec->commands)) {
+ if (!hasRegionOverflowed(osec->memRegion) &&
+ !hasRegionOverflowed(osec->lmaRegion))
+ break;
+
+ auto *isd = dyn_cast<InputSectionDescription>(cmd);
+ if (!isd)
+ continue;
+ for (InputSection *isec : reverse(isd->sections)) {
+ // Potential spill locations cannot be spilled.
+ if (isa<PotentialSpillSection>(isec))
+ continue;
+
+ // Find the next spill location.
+ auto it = potentialSpillLists.find(isec);
+ if (it == potentialSpillLists.end())
+ continue;
+
+ spilled = true;
+ PotentialSpillList &list = it->second;
+
+ PotentialSpillSection *spill = list.head;
+ if (!spill->next)
+ potentialSpillLists.erase(isec);
+ else
+ list.head = spill->next;
+
+ spilledInputSections.insert(isec);
+
+ // Replace the next spill location with the spilled section and adjust
+ // its properties to match the new location.
+ *llvm::find(spill->isd->sections, spill) = isec;
+ isec->parent = spill->parent;
+
+ // The alignment of the spill section may have diverged from the
+ // original due to e.g. a SUBALIGN. Correct assignment requires the
+ // spill's alignment to be used, not the original.
+ isec->addralign = spill->addralign;
+
+ // Record the (potential) reduction in the region's end position.
+ osec->memRegion->curPos -= isec->getSize();
+ if (osec->lmaRegion)
+ osec->lmaRegion->curPos -= isec->getSize();
+
+ // Spilling continues until the end position no longer overflows the
+ // region. Then, another round of address assignment will either confirm
+ // the spill's success or lead to yet more spilling.
+ if (!hasRegionOverflowed(osec->memRegion) &&
+ !hasRegionOverflowed(osec->lmaRegion))
+ break;
+ }
+
+ // Remove any spilled input sections to complete their move.
+ if (!spilledInputSections.empty())
----------------
mysterymath wrote:
Done.
https://github.com/llvm/llvm-project/pull/90007
More information about the llvm-commits
mailing list