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

Add support for Zilsd and Zcmlsd extension #117

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

lavanyajagan
Copy link
Collaborator

@lavanyajagan lavanyajagan commented Jun 18, 2024

Description

Adding Support for Zilsd and Zcmlsd Extensions presented here:

Related Issues

The related PRs

Update to/for Ratified/Unratified Extensions or to framework

  • Ratified
  • Unratified
  • Framework

List Extensions

The unratified spec document:
https://github.com/riscv/riscv-zilsd/blob/main/zilsd.adoc

Reference Model Used

  • SAIL
  • Spike
  • Other - < SPECIFY HERE >

Mandatory Checklist:

  • Make sure to have created a suitable entry in the CHANGELOG.md under [WIP-DEV] section.

@lavanyajagan lavanyajagan changed the title [Draft] Add support for Zilsd and Zcmlsd extension Add support for Zilsd and Zcmlsd extension Jul 5, 2024
@UmerShahidengr
Copy link
Collaborator

@lavanyajagan have you generated the ACTs based on these macros? If yes, then can you please provide those ACTs for review. Since this is unratified, and alot of work is pending for approval, it will be difficult to recreate the whole environment at my side and review this work.
ACTs will be easier to review.

@lavanyajagan
Copy link
Collaborator Author

ARCH test PR:

I will check with James to remove the "Draft". Could we provide a docker environment of the whole setup for you to re-run at your end?

@UmerShahidengr
Copy link
Collaborator

@lavanyajagan I have found the tests in riscv-arch-tests (PR#467), for some reason, you have declared the tests in a draft PR. When will you release it for official review? It seems like the draft email will keep your other ones in the queue.

@lavanyajagan
Copy link
Collaborator Author

The arch-test PR is updated. Also merge conflicts are resolved in this PR. Kindly review

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