Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 8ded2a8

Browse files
committedMar 18, 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 75530e9 commit 8ded2a8

File tree

1 file changed

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

1 file changed

+7
-1
lines changed
 

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

+7-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

0 commit comments

Comments
 (0)
Please sign in to comment.