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

Bump nightly to 2024-10-21 #476

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kaspar030
Copy link
Collaborator

@kaspar030 kaspar030 commented Oct 21, 2024

Description

Issues/PRs references

Open Questions

Change checklist

  • I have cleaned up my commit history and squashed fixup commits.
  • I have followed the Coding Conventions.
  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.

@ROMemories
Copy link
Collaborator

nit: fix(runqueue): uptdate RunQueueIter impl lifetimes to current nightly: s/uptdate/update/

Copy link
Collaborator

@ROMemories ROMemories left a comment

Choose a reason for hiding this comment

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

Testing with the hello-world example:

  • espressif-esp32-c6-devkitc-1 works as expected
  • rpi-pico-w does not compile, there is another options(noreturn) to remove (I'm not sure why CI is not catching that)
  • nrf52840dk compiles but hard-faults in hardware

Haven't tested other examples or other architectures.

@@ -102,7 +102,6 @@ unsafe extern "C" fn PendSV() {
bx LR
",
sched = sym sched,
options(noreturn)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I couldn't easily find the documentation for this removed option, but I assume this is correct

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