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

Remove use of the start feature #750

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

tgross35
Copy link
Contributor

#![feature(start)] was removed in 1, but we make use of it in the intrinsics example. Replace use of this feature with #[no_mangle] applied to #[main].

We don't actually run this example so it is not a problem if this is not entirely accurate. Currently the example does not run to completion, instead invoking rust_begin_unwind.

@tgross35 tgross35 force-pushed the remove-start-feature branch from f0320f9 to 5ae9411 Compare January 27, 2025 13:07
`#![feature(start)]` was removed in [1], but we make use of it in the
intrinsics example. Replace use of this feature with `#[no_mangle]`
applied to `#[main]`.

We don't actually run this example so it is not a problem if this is not
entirely accurate. Currently the example does not run to completion,
instead invoking `rust_begin_unwind`.

[1]: rust-lang/rust#134299
@tgross35 tgross35 force-pushed the remove-start-feature branch from 5ae9411 to d40d61f Compare January 28, 2025 02:02
@tgross35 tgross35 enabled auto-merge (rebase) January 28, 2025 02:04
@tgross35 tgross35 merged commit 44ac940 into rust-lang:master Jan 28, 2025
26 checks passed
@tgross35 tgross35 deleted the remove-start-feature branch January 28, 2025 04:51
Sign up for free to 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.

1 participant