authorgravatar for loic@gregoire.devLoïc Grégoire <loic@gregoire.dev> 2026-10-04 21:30:10-04:00
committergravatar for mlugg@mlugg.co.ukMatthew Lugg <mlugg@mlugg.co.uk> 2026-10-16 12:41:30+01:00
logbdd6bc06ca5f9b4cc6dc141470a6efe1ed7e1f6c
tree83e573f8ef0e65cd76c6a1c0c00c21268a661932
parent7a6eb167bcde3ed8fdc81f88bd683b30b6290e45
signature Commit is signed but in an unrecognized format.

Elf2: keep track of nodeless symbols

Instead of iterating through *all* symbols, instead keep track of symbols without a node and iterate through that. The downside is that e.g TBSS and BSS symbols are mixed together, but the overall iteration size is much smaller than before.

1 files changed, 25 insertions(+), 6 deletions(-)

src/link/Elf2.zig+25-6
......@@ -137,6 +137,9 @@ node_global_symbols: std.array_hash_map.Auto(MappedFile.Node.Index, Symbol.Globa
137137/// See the definition of `DsoGlobals`.
138138dso_globals: DsoGlobals,
139139
140/// Contains all defined global symbols that do not have a node.
141nodeless_globals: std.array_hash_map.Auto(Symbol.Global.Index, void),
142
140143// These are needed to find local symbols when a nodeless section moves.
141144bss_inputs: std.ArrayList(Node.InputIndex),
142145tbss_inputs: std.ArrayList(Node.InputIndex),
......@@ -2744,6 +2747,7 @@ fn addGlobalSymbol(elf: *Elf, opts: Symbol.Global.AddOptions) (error{
27442747 try elf.globals.ensureUnusedCapacity(gpa, 1);
27452748 try elf.globals_by_name.ensureUnusedCapacity(gpa, 1);
27462749 try elf.node_global_symbols.ensureUnusedCapacity(gpa, 1);
2750 try elf.nodeless_globals.ensureUnusedCapacity(gpa, 1);
27472751
27482752 try Section.Index.symtab.node(elf).ensureMinimumSize(gpa, &elf.mf, switch (elf.shdrPtr(.symtab)) {
27492753 inline else => |shdr, class| elf.targetLoad(&shdr.size) + @sizeOf(class.ElfN().Sym),
......@@ -2911,6 +2915,10 @@ fn addGlobalSymbol(elf: *Elf, opts: Symbol.Global.AddOptions) (error{
29112915 old_head_ptr.prev_in_node = .wrap(gsi);
29122916 }
29132917
2918 if (opts.node == .none and opts.shndx != .UNDEF) {
2919 elf.nodeless_globals.putAssumeCapacity(gsi, {});
2920 }
2921
29142922 if (force_local_bind) {
29152923 elf.moveDemotedGlobal(gsi);
29162924 }
......@@ -3373,6 +3381,11 @@ fn setGlobalSymbolValue(elf: *Elf, gsi: Symbol.Global.Index, new: struct {
33733381 assert(global_ptr.prev_in_node == .none);
33743382 }
33753383
3384 _ = elf.nodeless_globals.swapRemove(gsi);
3385 if (new.node == .none) {
3386 try elf.nodeless_globals.put(elf.base.comp.gpa, gsi, {});
3387 }
3388
33763389 global_ptr.symtab_index.ptr(elf).node = new.node;
33773390
33783391 const old_head: Symbol.Global.Index.Optional = old_head: {
......@@ -4770,6 +4783,7 @@ fn create(
47704783 .symbols = .empty,
47714784 .default_sym_vers = .empty,
47724785 },
4786 .nodeless_globals = .empty,
47734787 .bss_inputs = .empty,
47744788 .tbss_inputs = .empty,
47754789 .shstrtab = .{ .map = .empty },
......@@ -4854,6 +4868,7 @@ pub fn deinit(elf: *Elf) void {
48544868 elf.dso_globals.string_bytes.deinit(gpa);
48554869 elf.dso_globals.symbols.deinit(gpa);
48564870 elf.dso_globals.default_sym_vers.deinit(gpa);
4871 elf.nodeless_globals.deinit(gpa);
48574872 elf.bss_inputs.deinit(gpa);
48584873 elf.tbss_inputs.deinit(gpa);
48594874 elf.shstrtab.map.deinit(gpa);
......@@ -11627,15 +11642,19 @@ fn updateNobitsAddress(
1162711642
1162811643 // Update globals.
1162911644 // Since nobits sections have no nodes, we can't just iterate over `node_global_symbols`,
11630 // so instead filter manually.
11631 for (elf.globals.items, 0..) |global, global_index| {
11632 if (global.symtab_index.ptr(elf).node != .none) continue;
11633 const symbol: Symbol.Id = .global(@fromBackingInt(@intCast(global_index)));
11634 const symbol_shndx = switch (elf.symPtr(global.symtab_index)) {
11645 // so instead filter manually through nodeless symbols.
11646 for (elf.nodeless_globals.keys()) |gsi| {
11647 const symbol: Symbol.Id = .global(gsi);
11648
11649 const symbol_shndx = switch (elf.symPtr(symbol.index(elf))) {
1163511650 inline else => |sym| elf.targetLoad(&sym.shndx),
1163611651 };
1163711652 if (symbol_shndx != expected_shndx) continue;
11638 symbol.flushMoved(elf, symbol.value(elf) - old_symbol_base + new_symbol_base);
11653
11654 symbol.flushMoved(
11655 elf,
11656 symbol.value(elf) - old_symbol_base + new_symbol_base,
11657 );
1163911658 }
1164011659
1164111660 Symbol.Id.local(shndx.get(elf).lsi).flushMoved(