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

xtensa: enable multilib for dc233c and sample_controller #701

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dcpleung
Copy link
Member

@dcpleung dcpleung commented Aug 31, 2023

This enables multilib support for building GCC. This is needed to support both CALL0 and Windowed ABIs.

Needs: zephyrproject-rtos/gcc#23

Copy link
Contributor

@andyross andyross left a comment

Choose a reason for hiding this comment

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

Magnificent

@dcpleung
Copy link
Member Author

dcpleung commented Aug 31, 2023

If CI is building this correctly, we should be able to grab the artifacts in https://github.com/zephyrproject-rtos/sdk-ng/actions/runs/6043012028

CI actions won't produce correct binaries as it doesn't pull the GCC patch.

@dcpleung dcpleung force-pushed the xtensa/multilib branch 2 times, most recently from 32fb043 to 5a1c2eb Compare September 1, 2023 19:18
@stephanosio stephanosio self-requested a review November 15, 2023 05:24
This enables multilib support for building GCC. This is needed
to support both CALL0 and Windowed ABIs.

The corresponding GCC changes are needed to support multilib
where Windowed ABI is the default.

Signed-off-by: Daniel Leung <[email protected]>
@nashif
Copy link
Member

nashif commented Aug 15, 2024

@dcpleung why is this still a fraft? Can you rebase?

@dcpleung
Copy link
Member Author

dcpleung commented Aug 15, 2024

This is needed for CALL0 API support, and also to aid in adding CALL0 support code. Since we still have no viable CALL0 code to test, there is not much point in merging it now. That's why it's in draft until we have something to test with.

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.

3 participants