Skip to content

Update no_std test#264

Merged
tkaitchuck merged 2 commits intomasterfrom
tkaitchuck-patch-1
May 4, 2025
Merged

Update no_std test#264
tkaitchuck merged 2 commits intomasterfrom
tkaitchuck-patch-1

Conversation

@tkaitchuck
Copy link
Owner

No description provided.

@tkaitchuck tkaitchuck requested a review from Copilot May 4, 2025 06:05
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request updates the entry point of a bare-bones no_std application. Key changes include adding the no_main attribute, removing multiple experimental feature flags, and replacing #[start] with #[no_mangle] on the main function.

Comments suppressed due to low confidence (2)

no_std_test/src/main.rs:5

  • The removal of experimental features (start, core_intrinsics, lang_items, link_cfg) suggests an intentional change in the entry point configuration. Please ensure that with #[no_main] and #[no_mangle] the binary provides all necessary language items and is linked correctly for your no_std application.
#![feature(alloc_error_handler)]

no_std_test/src/main.rs:18

  • Replacing #[start] with #[no_mangle] changes how the entry point is defined and recognized. Verify that the new main function signature meets the requirements of your no_std environment and linker expectations.
#[no_mangle]

@tkaitchuck tkaitchuck merged commit 2228126 into master May 4, 2025
18 checks passed
@tkaitchuck tkaitchuck deleted the tkaitchuck-patch-1 branch May 6, 2025 17:09
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.

2 participants