Skip to content

Conversation

@ved-rivos
Copy link
Contributor

  1. The encoding.h could not be generated due to a build issue in riscv-opcodes. Will replace with auto-generated encoding.h after the issue is resolved.
  2. Extension is in draft state.

Copy link
Collaborator

@aswaterman aswaterman left a comment

Choose a reason for hiding this comment

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

LGTM, but we'll leave this as an open PR until it appears that it'll freeze.

@aswaterman
Copy link
Collaborator

@ved-rivos I think it's safe to incorporate this now. Can you rebase and fix the merge conflicts?

@ved-rivos
Copy link
Contributor Author

ved-rivos commented Nov 13, 2025

@aswaterman - could you please merge the riscv-opcodes PR riscv/riscv-opcodes#349 so I can use the encoding.h from master for this PR.

@aswaterman
Copy link
Collaborator

Done.

@ved-rivos
Copy link
Contributor Author

@aswaterman - this is rebased and ready.

@aswaterman
Copy link
Collaborator

Cool, thanks.

@aswaterman aswaterman merged commit c4524fc into riscv-software-src:master Nov 14, 2025
3 checks passed
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