Uh oh!
There was an error while loading. Please reload this page.
Validate that naked functions are never inlined - #87652
Merged
Merged
Conversation
rust-highfive
commented
Jul 31, 2021
Contributor
r? @oli-obk (rust-highfive has picked a reviewer for you, use r? to override) |
15 tasks
This comment has been minimized.
This comment has been minimized.
josephlr
reviewed
Aug 1, 2021
Contributor
There was a problem hiding this comment.
The tracking issue says that this should be an error, but it's a warning in this PR. Is this just a transitional lint to avoid breaking nightly code? If so, seems reasonable to me.
ContributorAuthor
There was a problem hiding this comment.
@josephlr There's a bunch of these lints that should be errors according to the RFC. I wanted to get them all in as warnings first and then file a follow-up which switches the relevant ones to errors. This allows us to reason about them as an entire set.
oli-obk
commented
Aug 2, 2021
Contributor
r? @Amanieu |
Amanieu
reviewed
Aug 2, 2021
Uh oh!
There was an error while loading. Please reload this page.
This comment has been minimized.
This comment has been minimized.
Reject all uses of the inline attribute on naked functions. rust-lang/rfcs#2774rust-lang/rfcs#2972
Amanieu
commented
Aug 3, 2021
Member
@bors r+ |
bors
commented
Aug 3, 2021
Collaborator
📌 Commit 157e0a0 has been approved by |
bors added a commit
to rust-lang-ci/rust
that referenced
this pull request
Aug 3, 2021
Rollup of 8 pull requests Successful merges: - rust-lang#87645 (Properly find owner of closure in THIR unsafeck) - rust-lang#87646 (Fix a parser ICE on invalid `fn` body) - rust-lang#87652 (Validate that naked functions are never inlined) - rust-lang#87685 (Write docs for SyncOnceCell From and Default impl) - rust-lang#87693 (Add `aarch64-apple-ios-sim` as a possible target to the manifest) - rust-lang#87708 (Add convenience method for handling ipv4-mapped addresses by canonicalizing them) - rust-lang#87711 (Correct typo) - rust-lang#87716 (Allow generic SIMD array element type) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
npmccallum added a commit
to npmccallum/rust-rfcs
that referenced
this pull request
Aug 4, 2021
This matches the behavior of the merged PR implementing this criteria. rust-lang/rust#87652
tmiasko added a commit
to tmiasko/rust
that referenced
this pull request
Jan 21, 2022
Transition unsupported naked functions future incompatibility lint into an error: * Naked functions must contain a single inline assembly block. Introduced as future incompatibility lint in 1.50 rust-lang#79653. Change into an error fixes a soundness issue described in rust-lang#32489. * Naked functions must not use any forms of inline attribute. Introduced as future incompatibility lint in 1.56 rust-lang#87652.
matthiaskrgr added a commit
to matthiaskrgr/rust
that referenced
this pull request
Jan 22, 2022
…nctions, r=Amanieu Reject unsupported naked functions Transition unsupported naked functions future incompatibility lint into an error: * Naked functions must contain a single inline assembly block. Introduced as future incompatibility lint in 1.50 rust-lang#79653. Change into an error fixes a soundness issue described in rust-lang#32489. * Naked functions must not use any forms of inline attribute. Introduced as future incompatibility lint in 1.56 rust-lang#87652. Closesrust-lang#32490. Closesrust-lang#32489. r? `@Amanieu` `@npmccallum` `@joshtriplett`
matthiaskrgr added a commit
to matthiaskrgr/rust
that referenced
this pull request
Jan 22, 2022
…nctions, r=Amanieu Reject unsupported naked functions Transition unsupported naked functions future incompatibility lint into an error: * Naked functions must contain a single inline assembly block. Introduced as future incompatibility lint in 1.50 rust-lang#79653. Change into an error fixes a soundness issue described in rust-lang#32489. * Naked functions must not use any forms of inline attribute. Introduced as future incompatibility lint in 1.56 rust-lang#87652. Closesrust-lang#32490. Closesrust-lang#32489. r? ``@Amanieu`` ``@npmccallum`` ``@joshtriplett``
matthiaskrgr added a commit
to matthiaskrgr/rust
that referenced
this pull request
Jan 22, 2022
…nctions, r=Amanieu Reject unsupported naked functions Transition unsupported naked functions future incompatibility lint into an error: * Naked functions must contain a single inline assembly block. Introduced as future incompatibility lint in 1.50 rust-lang#79653. Change into an error fixes a soundness issue described in rust-lang#32489. * Naked functions must not use any forms of inline attribute. Introduced as future incompatibility lint in 1.56 rust-lang#87652. Closesrust-lang#32490. Closesrust-lang#32489. r? ```@Amanieu``` ```@npmccallum``` ```@joshtriplett```
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reject all uses of the inline attribute on naked functions.
rust-lang/rfcs#2774
rust-lang/rfcs#2972
cc @joshtriplett@tmiasko@Amanieu