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

[ACT] [CTG] [ISAC] Add support for Zcf and Zcd extension #497

Merged
merged 8 commits into from
Dec 30, 2024

Conversation

anuani21
Copy link
Contributor

@anuani21 anuani21 commented Sep 26, 2024

Description

  • Add test case for compressed single precision floating point load and store operation.
  • Add test case for compressed double precision floating point load and store operation.

Related Issues

  • NA

Ratified/Unratified Extensions

[✓] Ratified

List Extensions

- Zcf
- Zcd

Reference Model Used

[✓] Spike

Mandatory Checklist:

Ran reports are placed here

For RV32Zcf:
https://gitlab.com/ptprasanna/actreports/-/tree/main/RV32Zcf?ref_type=heads
For RV32Zcd:
https://gitlab.com/ptprasanna/actreports/-/tree/main/RV32Zcf?ref_type=heads
For RV64Zcd:
https://gitlab.com/ptprasanna/actreports/-/tree/main/RV64Zcd?ref_type=heads

@jamesbeyond jamesbeyond changed the title Add support for Zcf and Zcd extension [ACT] [CTG] [ISAC] Add support for Zcf and Zcd extension Dec 15, 2024
Copy link
Collaborator

@UmerShahidengr UmerShahidengr left a comment

Choose a reason for hiding this comment

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

Looks good to go

@UmerShahidengr UmerShahidengr merged commit 33d1289 into riscv-non-isa:dev Dec 30, 2024
2 checks passed
@jordancarlin
Copy link
Contributor

Is there a reason the Zcd tests were only added for rv32i and not also for rv64i?

@anuani21
Copy link
Contributor Author

By mistake I didnt added RV64 tesct cases for Zcd extension.So i have opened a new PR [ACT] [CTG] [ISAC] Add support Zcd extension in RV64 #587.

@UmerShahidengr,can you please review it and give approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants