bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

tests: accept some noise from LLVM 21 in symbols-all-mangled

Browse source 
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
This commit is contained in:
Augie Fackler 2025-03-18 11:11:51 -04:00
parent 87e60a7d28
commit cafd23896d
1 changed files with 14 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

16
tests/run-make/symbols-all-mangled/rmake.rs
View file

@ -41,7 +41,13 @@ fn symbols_check_archive(path: &str) {
continue; // Unfortunately LLVM doesn't allow us to mangle this symbol
}
panic!("Unmangled symbol found: {name}");
if name.contains(".llvm.") {
// Starting in LLVM 21 we get various implementation-detail functions which
// contain .llvm. that are not a problem.
continue;
}
panic!("Unmangled symbol found in {path}: {name}");
}
}
@ -75,7 +81,13 @@ fn symbols_check(path: &str) {
continue; // Unfortunately LLVM doesn't allow us to mangle this symbol
}
panic!("Unmangled symbol found: {name}");
if name.contains(".llvm.") {
// Starting in LLVM 21 we get various implementation-detail functions which
// contain .llvm. that are not a problem.
continue;
}
panic!("Unmangled symbol found in {path}: {name}");
}
}