Add Unratified Zvzip extension instructions #1449
Open
+289
−12
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ved-rivos
force-pushed
the
zvzip
branch
3 times, most recently
from
8a135fc to
ef746cc
Compare
Collaborator
|
Is there a reason the spec is in your personal GitHub fork and not an official RISC-V International repo? Which TG is this under? |
Contributor
Author
|
Its a fast track extension. Presently the spec is held at that link for convenience. It will be turned into a PR against the ISA manual. |
ved-rivos
force-pushed
the
zvzip
branch
3 times, most recently
from
9bf6b8f to
53bae9b
Compare
nadime15
reviewed
Jan 5, 2026
nadime15
reviewed
Jan 6, 2026
Collaborator
nadime15
left a comment
nadime15
left a comment
There was a problem hiding this comment.
I have not fully reviewed it yet, as I think there are already more than enough comments for now. Once those are addressed, I will continue the review.
In addition, please follow the exact style used in the other vector related files (for example, vext_arith_insts.sail), with focus on alignment, variable naming, and program flow (this includes checking for invalid configurations (such as overlaps) before reading any variables).
ved-rivos
force-pushed
the
zvzip
branch
2 times, most recently
from
f10de1a to
5aca6df
Compare
ved-rivos
force-pushed
the
zvzip
branch
2 times, most recently
from
c08efc7 to
65d34c5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Spec: riscv/riscv-isa-manual#2529