diff options
| author | Augie Fackler <augie@google.com> | 2025-03-18 11:11:51 -0400 |
|---|---|---|
| committer | Augie Fackler <augie@google.com> | 2025-03-20 11:35:59 -0400 |
| commit | cafd23896db923e5fc83a81bb0226307940e4dde (patch) | |
| tree | 0aac61ec9f1975e3beb30720bbfb3fb7155cbd52 | |
| parent | 87e60a7d285f8ea560189024f85f2b194d1f168f (diff) | |
| download | rust-cafd23896db923e5fc83a81bb0226307940e4dde.tar.gz rust-cafd23896db923e5fc83a81bb0226307940e4dde.zip | |
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
| -rw-r--r-- | tests/run-make/symbols-all-mangled/rmake.rs | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/tests/run-make/symbols-all-mangled/rmake.rs b/tests/run-make/symbols-all-mangled/rmake.rs index 81f2678e55c..1fb03c62399 100644 --- a/tests/run-make/symbols-all-mangled/rmake.rs +++ b/tests/run-make/symbols-all-mangled/rmake.rs @@ -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}"); } } |
