Repository navigation
fix rustc build with llvm head #96362
Description
Activity
There's a patch for this at #96215.
Reacted by Nico Weber- added a commit that references this issue
on Apr 24, 2022 It builds now, but at least on our bots, src/test/ui/numeric/numeric-cast.rs fails with trunk LLVM: https://ci.chromium.org/ui/p/chromium/builders/try/linux_upload_clang/2200/overview => https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket/8815915618951615473/+/u/package_rust/stdout?format=raw
{"message":"aborting due to 113 previous errors","code":null,"level":"error","spans":[],"children":[],"rendered":"error: aborting due to 113 previous errors\n\n"} {"message":"For more information about this error, try `rustc --explain E0308`.","code":null,"level":"failure-note","spans":[],"children":[],"rendered":"For more information about this error, try `rustc --explain E0308`.\n"} thread '[ui] src/test/ui/numeric/numeric-cast.rs' panicked at 'explicit panic', src/tools/compiletest/src/json.rs:121:21 failures: [ui] src/test/ui/numeric/numeric-cast.rs(And many more lines of output before that.)
#96229 is specifically about the interaction with JSON parsing that makes the failure extra non-obvious.
Hm, someone on #94322 says "#94327 should help with this specific instance", but that's been merged a while ago. So I guess #94322 covers something more general than the specific instance fixed by that change.
If the test doesn't consistently pass, the failure is understood, and a fix won't happen soon, maybe that test should be disabled for now?
@chbaker0 too :)
Opened #96551 to try and remove the nondeterminism based on the path length, which should alleviate the problem (it fixed the
numeric-cast.rstests at least).- added a commit that references this issue
on Feb 3, 2026
building rust started failing like so:
Need to update rust code to work after https://reviews.llvm.org/D124216 (createAddressSanitizerFunctionPass, createModuleAddressSanitizerLegacyPassPass removal, https://reviews.llvm.org/D123882 populateThinLTOPassManager removal).
6234a56 has a related example.
@nikic Do you know if Augie (or anyone else) has a patch for this already?
@adetaylor