Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Make check{2,3} faster by enabling parallel rustc for dcou#34739

Merged
ryoqun merged 1 commit intosolana-labs:masterfrom
ryoqun:defend-dcou-from-long-ci
Jan 11, 2024
Merged

Make check{2,3} faster by enabling parallel rustc for dcou#34739
ryoqun merged 1 commit intosolana-labs:masterfrom
ryoqun:defend-dcou-from-long-ci

Conversation

@ryoqun
Copy link
Copy Markdown
Contributor

@ryoqun ryoqun commented Jan 11, 2024

Problem

dcou needs some bruteforce compilation for every crates to circumvent cargo's feature unification for its intended functioning. so, it's slow. and people (including me) are starting to be frustrated about it.

Summary of Changes

use recent rust's parallelization for front-end. this is especially effective, because the dcou check only doesn't care about generating binaries.

https://blog.rust-lang.org/2023/11/09/parallel-rustc.html

results (15%-25% faster):

before (taken from #34732):
buildkite/solana/checks2 Passed (7 minutes)
buildkite/solana/checks3 Passed (14 minutes, 50 seconds)

after:
buildkite/solana/checks2 Passed (5 minutes, 55 seconds)
buildkite/solana/checks3 Passed (10 minutes, 52 seconds)

related: #32169

@ryoqun ryoqun requested a review from yihau January 11, 2024 02:19
@ryoqun ryoqun changed the title Try to make check3 faster Make check2/check3 faster with parallel rustc for dcou Jan 11, 2024
@ryoqun ryoqun changed the title Make check2/check3 faster with parallel rustc for dcou Make check2&check3 faster with parallel rustc for dcou Jan 11, 2024
@ryoqun ryoqun changed the title Make check2&check3 faster with parallel rustc for dcou Make check2&check3 faster by enabling parallel rustc for dcou Jan 11, 2024
@ryoqun ryoqun changed the title Make check2&check3 faster by enabling parallel rustc for dcou Make check{2,3} faster by enabling parallel rustc for dcou Jan 11, 2024
Copy link
Copy Markdown
Contributor

@yihau yihau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you <3 just checked in my devbox. it save me 20~25% time 🤩 can't wait for this one to become stable

@ryoqun ryoqun merged commit a1be104 into solana-labs:master Jan 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants