Skip to content

Rollup of 7 pull requests - #77272

Merged
bors merged 18 commits into
rust-lang:masterfrom
jonas-schievink:rollup-dydo5kn
Sep 27, 2020
Merged

Rollup of 7 pull requests#77272
bors merged 18 commits into
rust-lang:masterfrom
jonas-schievink:rollup-dydo5kn

Conversation

@jonas-schievink

Copy link
Copy Markdown
Contributor

Successful merges:

Failed merges:

r? @ghost

ecstatic-morseand others added 18 commits September 25, 2020 14:31
This used to happen as a side-effect of `is_min_const_fn`, which was
subtle.
This should be caught by the new check in `rustc_passes`. At some point,
this function will be removed entirely.
This patch also:
* Add soft-float supports: only f32
* zero-extend i8/i16 to i32 because MIPS only supports register-length
arithmetic.
* Update table in asm! chapter in unstable book.
This mis-highlighted the entire documentation as code.
…separate lints
This is not ideal because it means `deny(broken_intra_doc_links)` will
no longer `deny(private_intra_doc_links)`. However, it can't be fixed
with a new lint group, because `broken` is already in the `rustdoc` lint
group; there would need to be a way to nest groups somehow.
This also removes the early `return` so that the link will be generated
even though it gives a warning.
Add asm! support for MIPS
For now, I only add support for mips32.
mips64 may come in future PRs if I could learn more about the target.
…r=oli-obk
Check for missing const-stability attributes in `rustc_passes`
Currently, this happens as a side effect of `is_min_const_fn`, which is non-obvious. Also adds a test for this case, since we didn't seem to have one before.
Separate `private_intra_doc_links` and `broken_intra_doc_links` into separate lints
This is not ideal because it means `deny(broken_intra_doc_links)` will
no longer `deny(private_intra_doc_links)`. However, it can't be fixed
with a new lint group, because `broken` is already in the `rustdoc` lint
group; there would need to be a way to nest groups somehow.
This also removes the early `return` so that the link will be generated
even though it gives a warning.
r? @Manishearth
cc @ecstatic-morse (#77242 (comment))
@jonas-schievink

Copy link
Copy Markdown
ContributorAuthor

@bors r+ rollup=never p=7

@rustbot modify labels: rollup

@bors

bors commented Sep 27, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 344ab3f has been approved by jonas-schievink

@rustbotrustbot added the rollup A PR which is a rollup label Sep 27, 2020
@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Sep 27, 2020
@bors

bors commented Sep 27, 2020

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 344ab3f with merge 4529af9...

@bors

bors commented Sep 27, 2020

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions, checks-azure
Approved by: jonas-schievink
Pushing 4529af9 to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Sep 27, 2020
@bors
bors merged commit 4529af9 into rust-lang:masterSep 27, 2020
@rustbotrustbot added this to the 1.48.0 milestone Sep 27, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.rollupA PR which is a rollupS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@jonas-schievink@bors@rustbot@ecstatic-morse@tshepang@tesuji@jyn514@bugadani