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

Split fnmadd_b15, fmadd_b15, fmsub_b15, and fnmsub_b15 in rv32i_m #465

Merged
merged 3 commits into from
May 24, 2024

Conversation

UmerShahidengr
Copy link
Collaborator

Description

This PR is the continuation of PR#461. In PR#461, fnmadd_b15, fmadd_b15, fmsub_b15, and fnmsub_b15 tests were split into 300 test files each, now these tests have been re-merged and split into 50 tests using this script. Each splited test has 768 tests (1286), and adding 1538 signature entries (2 entries per test, 768 tests implies 7682=1538 entries). Sail log file per test is now 2.1MB and consists of ~96k lines. Each split-test has been saved into a separate directory to make it easier to access and run.

Ratified/Unratified Extensions

  • Ratified
  • Unratified

List Extensions

RV32IF_Zicsr,RV32IFD_Zicsr,RV64IF_Zicsr,RV64IFD_Zicsr,RV32EF_Zicsr,RV32EFD_Zicsr,RV64EF_Zicsr,RV64EFD_Zicsr

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

@allenjbaum
Copy link
Collaborator

I see only files being deleted; I see nothing being added. What am I missing here?

@jamesbeyond jamesbeyond added the WorkRequired Not completed yet and requires further work label May 20, 2024
@UmerShahidengr UmerShahidengr changed the base branch from main to dev May 24, 2024 04:20
@UmerShahidengr UmerShahidengr changed the base branch from dev to main May 24, 2024 04:43
@UmerShahidengr UmerShahidengr changed the base branch from main to dev May 24, 2024 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ratified ratified specs unprivileged WorkRequired Not completed yet and requires further work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants