Skip to content

rv64g initial import #364

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

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

rv64g initial import #364

wants to merge 2 commits into from

Conversation

ct-clmsn
Copy link

@ct-clmsn ct-clmsn commented Aug 9, 2025

generic risc-v support for g (short hand for the imafdc base extensions)

@insertinterestingnamehere
Copy link
Collaborator

Thanks! We're certainly interested in having this. This is a fairly critical section of the code though, so review will take a bit longer.

@ct-clmsn
Copy link
Author

ct-clmsn commented Aug 12, 2025

Glad to hear there's interest. I'll double back on the smaller issues. The code passes make test on the jh7110 chips I've available. Do you all require floating point registers (and possibly vector registers) to be handled by the assembly that manages context switching?

@insertinterestingnamehere
Copy link
Collaborator

Great question! As best I can tell, the correct behavior for the various registers depends on the calling convention for the platform. From the perspective of the end-user, the context swap appears to happen inside an opaque function call. The context swap is supposed to save/restore whatever context is saved/restored by a callee and not cleared away by the caller. It doesn't have to worry about clearing away state that is saved/restored by the caller. I don't know offhand which case applies on RISC-V.

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