Commit 6c1a5a1e5b12e775552e1e3ff52daa6576789487 | zig

author	Loïc Grégoire <loic@gregoire.dev> 2026-10-12 14:29:40-04:00
committer	Matthew Lugg <mlugg@mlugg.co.uk> 2026-10-16 12:41:30+01:00
log	6c1a5a1e5b12e775552e1e3ff52daa6576789487
tree	019523796ece7d65c76abb8ec0e52c5ada41768f
parent	626381706712eb593bcf0c843dd2275d71cf41da
signature	Commit is signed but in an unrecognized format.

Elf2: allow custom NOBITS sections in objects

Elf2 previously was failing on objects with sections such as
'.bss.something', now preserve those sections for ET_REL output and
merge them into their respective .bss/.tbss for other output types.

1 files changed, 17 insertions(+), 17 deletions(-)

src
- link
  - Elf2.zig

src/link/Elf2.zig+17-17

 @@ -7090,7 +7090,7 @@ fn mapInputSection(elf: *Elf, opts: struct {
             .type = switch (opts.type) {
                 .PROGBITS => .NULL, // size is 0
                 .X86_64_UNWIND => .X86_64_UNWIND,
-                .NOBITS => return error.UnsupportedNobitsSection,
+                .NOBITS => if (elf.ehdrType() == .REL) .NOBITS else return error.UnsupportedNobitsSection,
                 else => unreachable,
             },
             .flags = flags: {
 @@ -7568,10 +7568,8 @@ fn loadObject(
                 shdr: ElfN.Shdr,
                 placement: union(enum) {
                     progbits: InputSection.Index,
-                    bss: struct {
+                    nobits: struct {
-                        offset: u64,
+                        shndx: Section.Index,
-                    },
-                    tbss: struct {
                         offset: u64,
                     },
                     none,
 @@ -7743,14 +7741,16 @@ fn loadObject(
                         },
                     };
+                    section.placement = .{ .nobits = .{
+                        .shndx = opts.shndx,
+                        .offset = offset,
+                    } };
                     if (opts.shndx == elf.shndx.bss) {
-                        section.placement = .{ .bss = .{ .offset = offset } };
                         input_has_bss = true;
                     } else if (opts.shndx == elf.shndx.tbss) {
-                        section.placement = .{ .tbss = .{ .offset = offset } };
                         input_has_tbss = true;
                     } else {
-                        unreachable;
+                        assert(elf.ehdrType() == .REL);
                     }
                     continue;
 @@ -7882,15 +7882,15 @@ fn loadObject(
                                 elf.getNodeShndx(isi.node(elf)),
                                 input_sym.value,
                             },
-                            .bss => |bss| .{
+                            .nobits => |nobits| .{
-                                .none,
-                                elf.shndx.bss,
-                                elf.shndx.bss.vaddr(elf) + bss.offset + input_sym.value,
-                            },
-                            .tbss => |tbss| .{
                                 .none,
-                                elf.shndx.tbss,
+                                nobits.shndx,
-                                elf.tbssBaseOffset() + tbss.offset + input_sym.value,
+                                (if (elf.ehdrType() == .REL)
+                                    @as(u64, 0)
+                                else if (nobits.shndx.flags(elf).TLS)
+                                    elf.tbssBaseOffset()
+                                else
+                                    nobits.shndx.vaddr(elf)) + nobits.offset + input_sym.value,
                             },
                             .none => unreachable,
                         };
 @@ -7960,7 +7960,7 @@ fn loadObject(
                             const loc_node = switch (loc_sec.placement) {
                                 .none => continue,
                                 .progbits => |isi| isi.node(elf),
-                                .bss, .tbss => return diags.failParse(
+                                .nobits => return diags.failParse(
                                     path,
                                     "relocations applying to BSS (SHT_NOBITS) sections are unsupported",
                                     .{},