Skip to content

Commit af9f182

Browse files
committedMar 20, 2025·
tests: accept some noise from LLVM 21 in symbols-all-mangled
I'm not entirely sure this is correct, but it doesn't feel obviously-wrong so I figured I'd just start by sending a PR rather than filing a bug and letting it linger. @rustbot label llvm-main
1 parent 87e60a7 commit af9f182

File tree

1 file changed

+13
-1
lines changed
  • tests/run-make/symbols-all-mangled

1 file changed

+13
-1
lines changed
 

‎tests/run-make/symbols-all-mangled/rmake.rs

+13-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,13 @@ fn symbols_check_archive(path: &str) {
4141
continue; // Unfortunately LLVM doesn't allow us to mangle this symbol
4242
}
4343

44-
panic!("Unmangled symbol found: {name}");
44+
if name.contains(".llvm.") {
45+
// Starting in LLVM 21 we get various implementation-detail functions which
46+
// contain .llvm. that are not a problem.
47+
continue;
48+
}
49+
50+
panic!("Unmangled symbol found in {path}: {name}");
4551
}
4652
}
4753

@@ -74,6 +80,12 @@ fn symbols_check(path: &str) {
7480
if name.contains("rust_eh_personality") {
7581
continue; // Unfortunately LLVM doesn't allow us to mangle this symbol
7682
}
83+
84+
if name.contains(".llvm.") {
85+
// Starting in LLVM 21 we get various implementation-detail functions which
86+
// contain .llvm. that are not a problem.
87+
continue;
88+
}
7789

7890
panic!("Unmangled symbol found: {name}");
7991
}

0 commit comments

Comments
 (0)
Please sign in to comment.