Skip to content

Commit

Permalink
Auto merge of #127390 - Oneirical:rough-testimation, r=<try>
Browse files Browse the repository at this point in the history
Migrate `raw-dylib-inline-cross-dylib` and `raw-dylib-custom-dlltool` `run-make` tests to rmake

Part of #121876 and the associated [Google Summer of Code project](https://blog.rust-lang.org/2024/05/01/gsoc-2024-selected-projects.html).

Please try:

try-job: x86_64-msvc
try-job: i686-mingw
  • Loading branch information
bors committed Jul 6, 2024
2 parents 51917e2 + e705973 commit f91bfe2
Show file tree
Hide file tree
Showing 5 changed files with 86 additions and 43 deletions.
2 changes: 0 additions & 2 deletions src/tools/tidy/src/allowed_run_make_makefiles.txt
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,7 @@ run-make/print-target-list/Makefile
run-make/prune-link-args/Makefile
run-make/raw-dylib-alt-calling-convention/Makefile
run-make/raw-dylib-c/Makefile
run-make/raw-dylib-custom-dlltool/Makefile
run-make/raw-dylib-import-name-type/Makefile
run-make/raw-dylib-inline-cross-dylib/Makefile
run-make/raw-dylib-link-ordinal/Makefile
run-make/raw-dylib-stdcall-ordinal/Makefile
run-make/redundant-libs/Makefile
Expand Down
11 changes: 0 additions & 11 deletions tests/run-make/raw-dylib-custom-dlltool/Makefile

This file was deleted.

25 changes: 25 additions & 0 deletions tests/run-make/raw-dylib-custom-dlltool/rmake.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
// Instead of using the default dlltool, the rust compiler can also accept a custom
// command file with the -C dlltool flag. This test uses it to compile some rust code
// with the raw_dylib Windows-exclusive feature, and checks that the output contains
// the string passed from the custom dlltool, confirming that the default dlltool was
// successfully overridden.
// See https://github.com/rust-lang/rust/pull/109677

//@ only-windows
//@ only-gnu
//@ needs-dlltool
// Reason: this test specifically checks the custom dlltool feature, only
// available on Windows-gnu.

use run_make_support::{diff, rustc};

fn main() {
let out = rustc()
.crate_type("lib")
.crate_name("raw_dylib_test")
.input("lib.rs")
.arg("-Cdlltool=script.cmd")
.run()
.stdout_utf8();
diff().expected_file("output.txt").actual_text("actual_text", out).run();
}
30 changes: 0 additions & 30 deletions tests/run-make/raw-dylib-inline-cross-dylib/Makefile

This file was deleted.

61 changes: 61 additions & 0 deletions tests/run-make/raw-dylib-inline-cross-dylib/rmake.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
// When we generate the import library for a dylib or bin crate, we should generate it
// for the symbols both for the current crate and all upstream crates. This allows for
// using the link kind `raw-dylib` inside inline functions successfully. This test checks
// that the import symbols in the object files match this convention, and that execution
// of the binary results in all function names exported successfully.
// See https://github.com/rust-lang/rust/pull/102988

//@ only-windows

use run_make_support::{cc, diff, is_msvc, llvm_objdump, run, rustc};

fn main() {
rustc()
.crate_type("dylib")
.crate_name("raw_dylib_test")
.input("lib.rs")
.arg("-Cprefer-dynamic")
.run();
rustc()
.crate_type("dylib")
.crate_name("raw_dylib_test_wrapper")
.input("lib_wrapper.rs")
.arg("-Cprefer-dynamic")
.run();
rustc().crate_type("bin").input("driver.rs").arg("-Cprefer-dynamic").run();
llvm_objdump()
.arg("--private-headers")
.input("driver.exe")
.run()
// Make sure we don't find an import to the functions we expect to be inlined.
.assert_stdout_not_contains("inline_library_function")
// Make sure we do find an import to the functions we expect to be imported.
.assert_stdout_contains("library_function");
if is_msvc() {
cc().arg("-c").out_exe("extern_1.obj").input("extern_1.c").run();
cc().arg("-c").out_exe("extern_2.obj").input("extern_2.c").run();
cc().input("extern_1.obj")
.arg("-link")
.arg("-dll")
.arg("-out:extern_1.dll")
.arg("-noimplib")
.run();
cc().input("extern_2.obj")
.arg("-link")
.arg("-dll")
.arg("-out:extern_2.dll")
.arg("-noimplib")
.run();
} else {
cc().arg("-v").arg("-c").out_exe("extern_1.obj").input("extern_1.c").run();
cc().arg("-v").arg("-c").out_exe("extern_2.obj").input("extern_2.c").run();
cc().input("extern_1.obj").out_exe("extern_1.dll").arg("-shared").run();
cc().input("extern_2.obj").out_exe("extern_2.dll").arg("-shared").run();
}
let out = run("driver").stdout_utf8();
diff()
.expected_file("output.txt")
.actual_text("actual_output", out)
.normalize(r#"\\r"#, "")
.run();
}

0 comments on commit f91bfe2

Please sign in to comment.