Skip to content

Commit

Permalink
Auto merge of #126206 - Kobzol:disable-libstdc++-version-check, r=Mar…
Browse files Browse the repository at this point in the history
…k-Simulacrum

Remove libstdc++ version check error

This keeps the error message from rust-lang/rust#125411, but removes the `exit(1)` call.

This PR is mostly a hotfix to unblock bootstrap benchmarks in rustc-perf.

However, I think that it might be better to just print a warning, in general. If the ABI version does not match, the build might or might not work locally (as we can see on rustc-perf, where it works even if the reported ABI is 7).

If it does not work (and **if** we can always recognize this during the LLVM wrapper build, instead of having some silent miscompilations), then the user will have to update their libstdc++ anyway, the error does not help them out on its own. So it should be enough to just provide a better error message, without blocking the build.

But I'm not adamant on that, I just want to unblock bootstrap benchmarks until we can find a way to update libstdc++ on the collector machine.

CC `@onur-ozkan`

r? `@Mark-Simulacrum`
  • Loading branch information
bors committed Jun 10, 2024
2 parents a51684a + 972ab52 commit 5340444
Showing 0 changed files with 0 additions and 0 deletions.

0 comments on commit 5340444

Please sign in to comment.