Commit c6f619d
committed
Remove zeroes binary from test_archive_non_objects
This test was added in emscripten-core#10300 to check if we can handle non-objects in
archives, especially rust metadata files that start with two leading
zeroes. LLVM's file magic identifier thinks files with two leading
zeroes are COFF files. But Rust metadata files used to start with two
leading zeroes too, resulting in an error in LLVM tools. So emscripten-core#10300
bypassed use of `llvm-nm` to avoid that.
We were still using `llvm-ranlib`, which also ran the LLVM magic
identifier when trying to create a symbol table for an object, but
`llvm-ranlib` so far has ignored those errors. But recently
llvm/llvm-project@a20168d
made them explicit errors, so we couldn't run `llvm-ranlib` anymore with
archives containing objects that start with two leading zeroes.
But this is not relevant anymore because Rust fixed their object file
format in rust-lang/rust#66235, so their files are not mistaken by LLVM
for COFF files anymore. So this PR fixes this test to not include a
binary with two leading zeros, while still testing archival of
non-object files.1 parent b22065b commit c6f619d
1 file changed
Lines changed: 1 addition & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7154 | 7154 | | |
7155 | 7155 | | |
7156 | 7156 | | |
7157 | | - | |
7158 | | - | |
7159 | | - | |
7160 | 7157 | | |
7161 | 7158 | | |
7162 | 7159 | | |
7163 | 7160 | | |
7164 | | - | |
| 7161 | + | |
7165 | 7162 | | |
7166 | 7163 | | |
7167 | 7164 | | |
| |||
0 commit comments