Skip to content

Add Zilsd/Zclsd support#636

Closed
Mingzhu Yan (trdthg) wants to merge 2 commits into
riscv:devfrom
trdthg:zilsd
Closed

Add Zilsd/Zclsd support#636
Mingzhu Yan (trdthg) wants to merge 2 commits into
riscv:devfrom
trdthg:zilsd

Conversation

@trdthg

@trdthg Mingzhu Yan (trdthg) commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

Description

rebase on #467. Compared to the original PR, I only modified a little isa constraints in cgf and generate tests again, so coverage should be fine. Test use RV32ICZilsd_Zclsd All test passed

Sail PR riscv/sail-riscv#765

Since zilsd/zclsd currently has no gcc support, I tested it with the gcc/binutils from nxp (the original PR seems to use clang).

Related Issues

NA

Ratified/Unratified Extensions

  • Ratified
  • Unratified

List Extensions

List the extensions that your PR affects. In case of unratified extensions, please provide a link to the spec draft that was referred to make this PR.

Reference Model Used

  • SAIL
  • Spike
  • Other - < SPECIFY HERE >

Mandatory Checklist:

  • All tests are compliant with the test-format spec present in this repo ?
  • Ran the new tests on RISCOF with SAIL/Spike as reference model successfully ?
  • Ran the new tests on RISCOF in coverage mode
  • Link to Google-Drive folder containing the new coverage reports (See this for more info): < SPECIFY HERE >

Optional Checklist:

  • Were the tests hand-written/modified ?
  • Have you run these on any hard DUT model ? Please specify name and provide link if possible in the description
  • If you have modified arch_test.h Please provide a detailed description of the changes in the Description section above.

Mingzhu Yan (trdthg) and others added 2 commits April 9, 2025 08:57
Co-authored-by: Lavanya J <35677601+lavanyajagan@users.noreply.github.com>
Co-authored-by: kumaransvyoma <140789916+kumaransvyoma@users.noreply.github.com>
@jamesbeyond

Copy link
Copy Markdown
Collaborator

ACT repository have been updated act4 as the default branch, if you would like to contribute this PR to ACT4, we will need to ask you to review and update this PR, according to guidelines on the act4 branch. For now, we have to close this PR. Thanks!

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