-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Rustup installation instructions to README #1421
Conversation
https://bjorn3.github.io/2023/10/31/progress-report-oct-2023.html describes the way to use the backend via |
Maybe we should also mention it in https://nnethercote.github.io/perf-book/build-configuration.html#minimizing-compile-times or https://nnethercote.github.io/perf-book/compile-times.html |
And also document this in the unstable book. |
Added more usage instructions based on the blog post. Also sent PRs to the perf book (nnethercote/perf-book#72). I'm not sure if the unstable book is the correct place for this, as it contains just general descriptions of compiler flags. I think that the Rustc book might be more appropriate. |
The actual text of the new section looks good to me. It feels a bit weird to have using precompiled versions from rustup first, then manual building and finally using precompiled versions from CI though. |
I agree. I think that it makes sense to have the easiest way first though. What order would you prefer? |
Maybe swap building it yourself and using the CI artifacts? You would need to reword using the CI artifacts for that as it is currently written as alternative for building yourself. |
Now that cranelift is distributed with Rustup, I find myself constantly searching for the rust-lang/rust#81746 PR. I think that it would be useful to also describe the Rustup installation/usage instructions here in README.
I tried to reorganize it a little bit. |
Thanks! |
Now that cranelift is distributed with Rustup, I find myself constantly searching for the rust-lang/rust#81746 PR. I think that it would be useful to also describe the Rustup installation/usage instructions here in README.