Skip to content

Commit b351d88

Browse files
authored
Unrolled build for rust-lang#138669
Rollup merge of rust-lang#138669 - durin42:llvm-21-anon-func-unmangled, r=bjorn3 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
2 parents a4a11ac + cafd238 commit b351d88

File tree

1 file changed

+14
-2
lines changed
  • tests/run-make/symbols-all-mangled

1 file changed

+14
-2
lines changed

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

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

@@ -75,7 +81,13 @@ fn symbols_check(path: &str) {
7581
continue; // Unfortunately LLVM doesn't allow us to mangle this symbol
7682
}
7783

78-
panic!("Unmangled symbol found: {name}");
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+
}
89+
90+
panic!("Unmangled symbol found in {path}: {name}");
7991
}
8092
}
8193

0 commit comments

Comments
 (0)