Skip to content

Revert "Upgrade to LLVM 16, again" #109622

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed

Conversation

compiler-errors
Copy link
Member

This PR reverts the 3 commits in #109474 until we can find out why it's causing CI to fail.

cc @nikic
r? @ehuss


Reverts "Increase array size in array-map.rs"

This reverts commit 58ac25b.

Reverts "Upgrade to LLVM 16"

This reverts commit 237f703.

Reverts "Update dist-x86_64-linux to GCC 8.5"

This reverts commit 079a801.

Reverts "Increase array size in array-map.rs"

This reverts commit 58ac25b.

Reverts "Upgrade to LLVM 16"

This reverts commit 237f703.

Reverts "Update dist-x86_64-linux to GCC 8.5"

This reverts commit 079a801.
@rustbot
Copy link
Collaborator

rustbot commented Mar 26, 2023

⚠️ Warning ⚠️

  • These commits modify submodules.

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Mar 26, 2023
@ehuss
Copy link
Contributor

ehuss commented Mar 26, 2023

See https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/Repeated.20failures.20related.20to.20LLVM.20tools.3F/near/344586849 for more discussion. I'm not convinced that the LLVM update itself was a problem, but more of an issue of the LLVM cache was cleared, and there was a struggle getting past the mingw bug from #108227. The most recent PR made it through, so perhaps things will settle to the status quo from before #109474. Will see if the next PR goes through.

@compiler-errors compiler-errors deleted the revert-109474 branch August 11, 2023 20:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants