Skip to content

Commit 7877883

Browse files
committed
Run the glibc run-make test in opt-dist
1 parent 75f8cc6 commit 7877883

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

Diff for: src/tools/compiletest/src/header.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1280,7 +1280,7 @@ pub fn llvm_has_libzstd(config: &Config) -> bool {
12801280
let stderr = String::from_utf8(output.stderr).ok()?;
12811281
let zstd_available = !stderr.contains("LLVM was not built with LLVM_ENABLE_ZSTD");
12821282

1283-
// We don't partiCOMPILETEST_ENABLE_OPT_DIST_TESTScularly need to clean the link up (so the previous commands could fail
1283+
// We don't particularly need to clean the link up (so the previous commands could fail
12841284
// in theory but won't in practice), but we can try.
12851285
std::fs::remove_file(lld_symlink_path).ok()?;
12861286

Diff for: src/tools/opt-dist/src/tests.rs

+1
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,7 @@ llvm-config = "{llvm_config}"
103103
"tests/incremental",
104104
"tests/mir-opt",
105105
"tests/pretty",
106+
"tests/run-make/glibc-symbols-x86_64-unknown-linux-gnu",
106107
"tests/ui",
107108
"tests/crashes",
108109
];

Diff for: tests/run-make/glibc-symbols-x86_64-unknown-linux-gnu/rmake.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ fn get_glibc_symbols(file: &Path) -> Vec<GlibcSymbol> {
8181

8282
// Uses llvm-objdump, because implementing this using the `object` crate is quite complicated.
8383
llvm_objdump()
84-
.arg("-T")
84+
.arg("--dynamic-syms")
8585
.arg(file)
8686
.run()
8787
.stdout_utf8()

0 commit comments

Comments
 (0)