Skip to content

[RISCV][WIP] Add assembler support for Zvma #132965

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

Closed
wants to merge 1 commit into from

Conversation

FantasqueX
Copy link
Contributor

No description provided.

@topperc
Copy link
Collaborator

topperc commented Mar 25, 2025

SiFive has patches for this we are planning to post soon.

@sequencer
Copy link

Hi, @topperc we are depending on this for our zvma SW smoke test development this week. 🫣
Is there any schedule for SiFive’s upstreaming plan?
If it’s soon in this week, we will drop this PR, and wait for your work. Otherwise we might miss the AME meeting presentation.
Another choice is asking @FantasqueX adding this as draft until your review or close it until SiFive’s upstreaming their patches.
WDYT?

@kasanovic
Copy link

The patch should use custom names, not Zvma, as the extension is not official RVIA spec.

@sequencer
Copy link

@kasanovic is fine to change the name to XSfame or some other names, what's your suggestion?

@topperc
Copy link
Collaborator

topperc commented Mar 26, 2025

Hi, @topperc we are depending on this for our zvma SW smoke test development this week. 🫣 Is there any schedule for SiFive’s upstreaming plan? If it’s soon in this week, we will drop this PR, and wait for your work. Otherwise we might miss the AME meeting presentation. Another choice is asking @FantasqueX adding this as draft until your review or close it until SiFive’s upstreaming their patches. WDYT?

I have posted our patch. #133031

@sequencer
Copy link

Appreciate your work!

@SpriteOvO SpriteOvO closed this Mar 26, 2025
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.

5 participants