Skip to content

Rollup of 8 pull requests - #73401

Closed
Dylan-DPC-zz wants to merge 21 commits into
rust-lang:masterfrom
Dylan-DPC-zz:rollup-2qqjfwo
Closed

Rollup of 8 pull requests#73401
Dylan-DPC-zz wants to merge 21 commits into
rust-lang:masterfrom
Dylan-DPC-zz:rollup-2qqjfwo

Conversation

@Dylan-DPC-zz

Copy link
Copy Markdown

Successful merges:

Failed merges:

r? @ghost

jonas-schievinkand others added 21 commits June 14, 2020 13:20
Apparently it got changed.
…i-obk
remove visit_terminator_kind from MIR visitor
For some reason, we had both `visit_terminator` and `visit_terminator_kind`. In contrast, for `Statement` we just have `visit_statement`. So this cleans things up by removing `visit_terminator_kind` and porting its users to `visit_terminator`.
Don't run generator transform when there's a TyErr
Not sure if this might cause any problems later on, but we shouldn't be hitting codegen or const eval for the produced MIR anyways, so it should be fine.
cc rust-lang#72685 (comment)
…tions, r=kinnison
Re-order correctly the sections in the sidebar
Before that, "trait implementations" and "implementors" titles in the sidebar were before "methods" for example. Which wasn't logical considering that the two sections come after in the "content".
r? @kinnison
…num-cpus, r=Mark-Simulacrum
Add more info to `x.py build --help` on default value for `-j JOBS`.
Use `Ipv4Addr::from<[u8; 4]>` when possible
Resolve this comment: rust-lang#73331 (comment)
Fix forge-platform-support URL
Apparently it got changed.
@Dylan-DPC-zz

Copy link
Copy Markdown
Author

network issue on my side

@Dylan-DPC-zz
Dylan-DPC-zz deleted the rollup-2qqjfwo branch June 16, 2020 13:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@Dylan-DPC-zz@jonas-schievink@tesuji@GuillaumeGomez@pnkfelix@ratijas@RalfJung@rnestler@Dylan-DPC