Draft of 1.24 announcement #228
Conversation
|
r? @aturon (rust_highfive has picked a reviewer for you, use r? to override) |
|
@rust-lang/core please review! |
| This is still not the end story for compiler performance generally, nor incremental compilation | ||
| specifically. We have a lot more work planned in the future. For example, another change | ||
| related to performance hit stable this release: | ||
| [`codegen-units` is now set to 16 by default](https://github.com/rust-lang/rust/pull/46910). |
There was a problem hiding this comment.
It might be worth mentioning that this will likely make builds faster, but final products slower.
_posts/2018-02-15-Rust-1.24.md
Outdated
| > the compiler, like the RLS and Clippy, in the future. | ||
|
|
||
| Finally, please take note of that `-preview` in the name: `rustfmt` is still not quite | ||
| at 1.0 yet. Some tweaks to the default styles are still occuring, though they're relatively |
_posts/2018-02-15-Rust-1.24.md
Outdated
|
|
||
| ### Library stabilizations | ||
|
|
||
| If you're a fan of `str::find`, used to find a given `char` inside of a `&str`, you'll be |
There was a problem hiding this comment.
Might want to put which is in front of used. I had to reread the sentence a few times thinking the wording was a typo
_posts/2018-02-15-Rust-1.24.md
Outdated
|
|
||
| #### Incremental compilation | ||
|
|
||
| Back in September, we blogged about [Incremental Compilation](https://blog.rust-lang.org/2016/09/08/incremental.html). |
There was a problem hiding this comment.
Note that this post is from September 2016 (!) already.
There was a problem hiding this comment.
Can we say anything about the expected performance wins?
There was a problem hiding this comment.
It very much depends on what you are compiling and what changes you made. "twice as fast" seems to be a good bet, looking at http://perf.rust-lang.org. For cargo check it might be different (better, hopefully) but we don't track that on perf.rlo yet.
_posts/2018-02-15-Rust-1.24.md
Outdated
| changed, which means that that second build is faster. | ||
|
|
||
| As of Rust 1.24, this is now [turned on by default](https://github.com/rust-lang/cargo/pull/4817). | ||
| This means that your builds should get faster! |
There was a problem hiding this comment.
We should mention that cargo check should be used to get the best possible edit-compile cycle; can reference previous release announcement about that. I don't think we've done enough advertising of this as the recommended workflow.
There was a problem hiding this comment.
Also, do we have documentation somewhere that says when cargo defaults to incr. comp. and when it doesn't?
There was a problem hiding this comment.
i don't believe we do
_posts/2018-02-15-Rust-1.24.md
Outdated
| `rustup update stable`, `rustfmt-preview` will be updated as well. | ||
|
|
||
| > This strategy is expected to be used for other developer tools that integrate tightly with | ||
| > the compiler, like the RLS and Clippy, in the future. |
There was a problem hiding this comment.
yeah, but we've never talked about it before, so i phrased it that way
_posts/2018-02-15-Rust-1.24.md
Outdated
| at 1.0 yet. Some tweaks to the default styles are still occuring, though they're relatively | ||
| minor. Once `rustfmt` hits 1.0, we'll be releasing a `rustfmt` component and | ||
| deprecating `rustfmt-preview`. Since this is the first major component we're distributing | ||
| this way, we wanted to clearly signal that it is still developing. |
There was a problem hiding this comment.
MIght want to say something about this will happen without any user interaction - rustup will handle it.
There was a problem hiding this comment.
Oh, and it's the second component - you also have to use rls-preview
There was a problem hiding this comment.
same as above; we've never talked about rls-preview
_posts/2018-02-15-Rust-1.24.md
Outdated
| deprecating `rustfmt-preview`. Since this is the first major component we're distributing | ||
| this way, we wanted to clearly signal that it is still developing. | ||
|
|
||
| For more, please check out [`rustfmt` on GitHub](https://github.com/rust-lang-nursery/rustfmt). |
There was a problem hiding this comment.
Might be good to say that if you've previously installed rustfmt with Cargo then you'll have to remove it, but that rustup should tell you about that.
|
updated to incorporate everyone's feedback |
No description provided.