Skip to content

Add Zicfiss extension tests#417

Open
ved-rivos wants to merge 15 commits intoriscv:devfrom
ved-rivos:zicfiss
Open

Add Zicfiss extension tests#417
ved-rivos wants to merge 15 commits intoriscv:devfrom
ved-rivos:zicfiss

Conversation

@ved-rivos
Copy link
Contributor

@ved-rivos ved-rivos commented Dec 17, 2023

Description

Implements tests for Zicfiss extension.
Tests sspush, sspopchk, ssamoswap, ssrdp, c_sspush, and c_sspopchk instructions in S and U mode
Specification: https://github.com/riscv/riscv-cfi/blob/main/cfi_backward.adoc

Related Issues

N/A

Ratified/Unratified Extensions

  • Ratified
  • Unratified

List Extensions

Zicfiss: https://github.com/riscv/riscv-cfi/blob/main/cfi_backward.adoc

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 >
  • Link to PR in RISCV-ISAC from which the reports were generated : < SPECIFY HERE >
  • Changelog entry created with a minor patch

Optional Checklist:

  • RISCV-V CTG PR link if tests were generated using it : Add unratified Zicfiss extension riscv-software-src/riscv-ctg#94
  • 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.

@jamesbeyond jamesbeyond changed the base branch from main to dev May 28, 2024 16:56
@ved-rivos ved-rivos changed the title Add unratified Zicfiss extension tests Add Zicfiss extension tests Jul 3, 2024
Signed-off-by: Umer Shahid <umer.shahid@10xengineers.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants