-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Rename asm! to llvm_asm! #68404
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Rename asm! to llvm_asm! #68404
Conversation
r? @estebank (rust_highfive has picked a reviewer for you, use r? to override) |
Did you mean to update the rustc guide? |
25c3859
to
564837b
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
I think that it makes sense for this to go in a release or two before the new inline assembly support goes in, so that there's a migration period for people using nightly. Also, it occurs to me that in the new version of We could start doing that in this commit: could you make |
☔ The latest upstream changes (presumably #68133) made this pull request unmergeable. Please resolve the merge conflicts. |
0cae77d
to
990946e
Compare
This PR is ready for review. I haven't actually deprecated |
This comment has been minimized.
This comment has been minimized.
@bors retry (yielding to rollup :P ) |
⌛ Testing commit 1cc521e with merge 31fc0ed22e8fa9aba6d088917e2c65305b773161... |
@bors retry (sorry but last one :P ) |
@bors p=1 |
☔ The latest upstream changes (presumably #70451) made this pull request unmergeable. Please resolve the merge conflicts. |
☀️ Test successful - checks-azure |
Changes: ```` remove redundant import rustup rust-lang#68404 rustup rust-lang#69644 rustup rust-lang#70344 Move verbose_file_reads to restriction move redundant_pub_crate to nursery readme: explain how to run only a single lint on a codebase Remove dependency on `matches` crate Move useless_transmute to nursery nursery group -> style Update for PR feedback Auto merge of rust-lang#5314 - ehuss:remove-git2, r=flip1995 Lint for `pub(crate)` items that are not crate visible due to the visibility of the module that contains them ```` Fixes rust-lang#70456
submodules: update clippy from 1ff81c1 to 70b93aa Changes: ```` remove redundant import rustup rust-lang#68404 rustup rust-lang#69644 rustup rust-lang#70344 Move verbose_file_reads to restriction move redundant_pub_crate to nursery readme: explain how to run only a single lint on a codebase Remove dependency on `matches` crate Move useless_transmute to nursery nursery group -> style Update for PR feedback Auto merge of rust-lang#5314 - ehuss:remove-git2, r=flip1995 Lint for `pub(crate)` items that are not crate visible due to the visibility of the module that contains them ```` Fixes rust-lang#70456
Changes: ```` remove redundant import rustup rust-lang/rust#68404 rustup rust-lang/rust#69644 rustup rust-lang/rust#70344 Move verbose_file_reads to restriction move redundant_pub_crate to nursery readme: explain how to run only a single lint on a codebase Remove dependency on `matches` crate Move useless_transmute to nursery nursery group -> style Update for PR feedback Auto merge of rust-lang#5314 - ehuss:remove-git2, r=flip1995 Lint for `pub(crate)` items that are not crate visible due to the visibility of the module that contains them ```` Fixes #70456
As per rust-lang/rfcs#2843, this PR renames
asm!
tollvm_asm!
. It also renames the compiler's internalInlineAsm
data structures toLlvmInlineAsm
in preparation for the newasm!
functionality specified in rust-lang/rfcs#2850.This PR doesn't actually deprecate
asm!
yet, it just makes it redirect tollvm_asm!
. This is necessary because we first need to update the submodules (in particular stdarch) to usellvm_asm!
.