diff options
| author | Jakub Konka <kubkon@jakubkonka.com> | 2023-10-05 00:34:45 +0200 |
|---|---|---|
| committer | Jakub Konka <kubkon@jakubkonka.com> | 2023-10-16 19:33:04 +0200 |
| commit | 04a8f217c651f56a2e185d87dd40c0a87ee78bc5 (patch) | |
| tree | 88c21a1678fa5be114b227d873c89a9e067888cb /src/link/Elf/Object.zig | |
| parent | 9a80ac0429b34541b91671246833c47143d1e4da (diff) | |
| download | zig-04a8f217c651f56a2e185d87dd40c0a87ee78bc5.tar.gz zig-04a8f217c651f56a2e185d87dd40c0a87ee78bc5.zip | |
elf: fix COMDAT deduping logic
Diffstat (limited to 'src/link/Elf/Object.zig')
| -rw-r--r-- | src/link/Elf/Object.zig | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/link/Elf/Object.zig b/src/link/Elf/Object.zig index 9baa5c07ed..dede48896e 100644 --- a/src/link/Elf/Object.zig +++ b/src/link/Elf/Object.zig @@ -130,7 +130,10 @@ fn initAtoms(self: *Object, elf_file: *Elf) !void { continue; } - const group_signature_off = try self.strings.insert(elf_file.base.allocator, group_signature); + // Note the assumption about a global strtab used here to disambiguate common + // COMDAT owners. + const gpa = elf_file.base.allocator; + const group_signature_off = try elf_file.strtab.insert(gpa, group_signature); const gop = try elf_file.getOrCreateComdatGroupOwner(group_signature_off); const comdat_group_index = try elf_file.addComdatGroup(); const comdat_group = elf_file.comdatGroup(comdat_group_index); @@ -138,7 +141,7 @@ fn initAtoms(self: *Object, elf_file: *Elf) !void { .owner = gop.index, .shndx = shndx, }; - try self.comdat_groups.append(elf_file.base.allocator, comdat_group_index); + try self.comdat_groups.append(gpa, comdat_group_index); }, elf.SHT_SYMTAB_SHNDX => @panic("TODO SHT_SYMTAB_SHNDX"), @@ -184,7 +187,6 @@ fn addAtom( atom.name_offset = try elf_file.strtab.insert(elf_file.base.allocator, name); atom.file_index = self.index; atom.input_section_index = shndx; - atom.flags.alive = true; self.atoms.items[shndx] = atom_index; if (shdr.sh_flags & elf.SHF_COMPRESSED != 0) { |