Skip to content

Commit 1e9d09d

Browse files
committed
rewrite symbols-include-type-name to rmake
1 parent 9d055b0 commit 1e9d09d

File tree

4 files changed

+12
-14
lines changed

4 files changed

+12
-14
lines changed

src/tools/tidy/src/allowed_run_make_makefiles.txt

-1
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,6 @@ run-make/staticlib-dylib-linkage/Makefile
174174
run-make/std-core-cycle/Makefile
175175
run-make/symbol-mangling-hashed/Makefile
176176
run-make/symbol-visibility/Makefile
177-
run-make/symbols-include-type-name/Makefile
178177
run-make/sysroot-crates-are-unstable/Makefile
179178
run-make/target-cpu-native/Makefile
180179
run-make/target-specs/Makefile

tests/run-make/cdylib-fewer-symbols/rmake.rs

-4
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,6 @@
44
// four such symbols are successfully hidden.
55
// See https://github.com/rust-lang/rust/pull/45710
66

7-
//FIXME(Oneirical): try it on windows, restore ignore
8-
// See https://github.com/rust-lang/rust/pull/46207#issuecomment-347561753
9-
//FIXME(Oneirical): I also removed cross-compile ignore since there is no binary execution
10-
117
use run_make_support::{dynamic_lib_name, llvm_readobj, rustc};
128

139
fn main() {

tests/run-make/symbols-include-type-name/Makefile

-9
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
// Method names used to be obfuscated when exported into symbols,
2+
// leaving only an obscure `<impl>`. After the fix in #30328,
3+
// this test checks that method names are successfully saved in the symbol list.
4+
// See https://github.com/rust-lang/rust/issues/30260
5+
6+
use run_make_support::{invalid_utf8_contains, rustc};
7+
8+
fn main() {
9+
rustc().crate_type("staticlib").emit("asm").input("lib.rs").run();
10+
// Check that symbol names for methods include type names, instead of <impl>.
11+
invalid_utf8_contains("lib.s", "Def");
12+
}

0 commit comments

Comments
 (0)