user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Rollup merge of #138669 - durin42:llvm-21-anon-func-unmangled, r=bjorn3

Browse source 
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
This commit is contained in:
Matthias Krüger 2025-03-21 15:48:55 +01:00 • committed by GitHub
commit 015df66ee2
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
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}");
}
}