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

arch: arm: cortex_m: add ip & lr to the clobber list #75607

Conversation

ithinuel
Copy link
Collaborator

@ithinuel ithinuel commented Jul 8, 2024

Calls to other function may clobber ip & lr too so these register need to be added to the clobberlist.

fixes #75586

nordic-krch
nordic-krch previously approved these changes Jul 9, 2024
Copy link
Contributor

@nordic-krch nordic-krch left a comment

Choose a reason for hiding this comment

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

Issue is fixed. Looks good.

stephanosio
stephanosio previously approved these changes Jul 9, 2024
@aescolar aescolar added this to the v3.7.0 milestone Jul 9, 2024
@aescolar aescolar added the bug The issue is a bug, or the PR is fixing a bug label Jul 9, 2024
Calls to other function may clobber ip & lr too so these register need to
be added to the clobberlist.
r3 is not actually used in z_arm_switch_to_main_no_multithreading so it is
also removed from the clobber list.

Signed-off-by: Wilfried Chauveau <[email protected]>
@ithinuel ithinuel dismissed stale reviews from stephanosio and nordic-krch via 31e0132 July 9, 2024 14:27
@ithinuel ithinuel force-pushed the fix-no-optimization-switch-to-main-thread branch from a140d64 to 31e0132 Compare July 9, 2024 14:27
@nashif nashif merged commit 1b820df into zephyrproject-rtos:main Jul 10, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ARM ARM (32-bit) Architecture bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No multithreading mode with CONFIG_NO_OPTIMIZATIONS=y fails to start (Cortex-M)
7 participants