summaryrefslogtreecommitdiffstats
path: root/src/test/run-make-fulldeps/issue-64153
diff options
context:
space:
mode:
Diffstat (limited to 'src/test/run-make-fulldeps/issue-64153')
-rw-r--r--src/test/run-make-fulldeps/issue-64153/Makefile26
-rw-r--r--src/test/run-make-fulldeps/issue-64153/downstream.rs6
-rw-r--r--src/test/run-make-fulldeps/issue-64153/upstream.rs6
3 files changed, 38 insertions, 0 deletions
diff --git a/src/test/run-make-fulldeps/issue-64153/Makefile b/src/test/run-make-fulldeps/issue-64153/Makefile
new file mode 100644
index 000000000..9c0c3fe6e
--- /dev/null
+++ b/src/test/run-make-fulldeps/issue-64153/Makefile
@@ -0,0 +1,26 @@
+-include ../tools.mk
+
+# `llvm-objdump`'s output looks different on windows than on other platforms.
+# It should be enough to check on Unix platforms, so:
+# ignore-windows
+
+# Staticlibs don't include Rust object files from upstream crates if the same
+# code was already pulled into the lib via LTO. However, the bug described in
+# https://github.com/rust-lang/rust/issues/64153 lead to this exclusion not
+# working properly if the upstream crate was compiled with an explicit filename
+# (via `-o`).
+#
+# This test makes sure that functions defined in the upstream crates do not
+# appear twice in the final staticlib when listing all the symbols from it.
+
+all:
+ $(RUSTC) --crate-type rlib upstream.rs -o $(TMPDIR)/libupstream.rlib -Ccodegen-units=1
+ $(RUSTC) --crate-type staticlib downstream.rs -Clto -Ccodegen-units=1 -o $(TMPDIR)/libdownstream.a
+ # Dump all the symbols from the staticlib into `syms`
+ "$(LLVM_BIN_DIR)"/llvm-objdump -t $(TMPDIR)/libdownstream.a > $(TMPDIR)/syms
+ # Count the global instances of `issue64153_test_function`. There'll be 2
+ # if the `upstream` object file got erronously included twice.
+ # The line we are testing for with the regex looks something like:
+ # 0000000000000000 g F .text.issue64153_test_function 00000023 issue64153_test_function
+ grep -c -e "[[:space:]]g[[:space:]]*F[[:space:]].*issue64153_test_function" $(TMPDIR)/syms > $(TMPDIR)/count
+ [ "$$(cat $(TMPDIR)/count)" -eq "1" ]
diff --git a/src/test/run-make-fulldeps/issue-64153/downstream.rs b/src/test/run-make-fulldeps/issue-64153/downstream.rs
new file mode 100644
index 000000000..e03704665
--- /dev/null
+++ b/src/test/run-make-fulldeps/issue-64153/downstream.rs
@@ -0,0 +1,6 @@
+extern crate upstream;
+
+#[no_mangle]
+pub extern "C" fn foo() {
+ print!("1 + 1 = {}", upstream::issue64153_test_function(1));
+}
diff --git a/src/test/run-make-fulldeps/issue-64153/upstream.rs b/src/test/run-make-fulldeps/issue-64153/upstream.rs
new file mode 100644
index 000000000..861a00298
--- /dev/null
+++ b/src/test/run-make-fulldeps/issue-64153/upstream.rs
@@ -0,0 +1,6 @@
+// Make this function extern "C", public, and no-mangle, so that it gets
+// exported from the downstream staticlib.
+#[no_mangle]
+pub extern "C" fn issue64153_test_function(x: u32) -> u32 {
+ x + 1
+}