Skip to content
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

Move asm! and global_asm! to core::arch #87227

Merged
merged 1 commit into from
Jul 19, 2021
Merged

Conversation

bstrie
Copy link
Contributor

@bstrie bstrie commented Jul 17, 2021

Follow-up to rust-lang/stdarch#1183 .

Implements the libs-api team decision from #84019 (comment) .

In order to not break nightly users, this PR also adds the newly-moved items to the prelude. However, a decision will need to be made before stabilization as to whether these items should remain in the prelude. I will file an issue for this separately.

Fixes #84019 .

r? @Amanieu

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 17, 2021
@rust-log-analyzer

This comment has been minimized.

@bstrie
Copy link
Contributor Author

bstrie commented Jul 18, 2021

The current build failure is because the documentation for these macros isn't showing up in core::arch for some reason. The items are there, and they can correctly be used only when you refer to them via the proper paths, but for whatever reason the generated arch module docs don't list them and they can't be found anywhere in search. They have doc comments. Investigating this.

@rust-log-analyzer

This comment has been minimized.

@bstrie bstrie changed the title Move asm! and global_asm! to core::arch (step 2) Move asm! and global_asm! to core::arch Jul 18, 2021
@Amanieu
Copy link
Member

Amanieu commented Jul 19, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Jul 19, 2021

📌 Commit f26fbe2 has been approved by Amanieu

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 19, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 19, 2021
…laumeGomez

Rollup of 8 pull requests

Successful merges:

 - rust-lang#86230 (Add --nocapture option to rustdoc)
 - rust-lang#87210 (Rustdoc accessibility: make the sidebar headers actual headers)
 - rust-lang#87227 (Move asm! and global_asm! to core::arch)
 - rust-lang#87236 (Simplify command-line argument initialization on unix)
 - rust-lang#87251 (Fix "item info" width)
 - rust-lang#87256 (Extend HIR-based WF checking to associated type defaults)
 - rust-lang#87259 (triagebot shortcut config)
 - rust-lang#87268 (Don't create references to uninitialized data in `List::from_arena`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 65b7aa9 into rust-lang:master Jul 19, 2021
@rustbot rustbot added this to the 1.55.0 milestone Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: 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.

Namespace the asm! macro
6 participants