Open jjscheel opened 1 year ago
Before we get started on this, I'd like to make sure we complete the SOW. @ptprasanna, can you propose some text and place in a comment for review?
Majority of the codes changes are done as part of Zfinx, so have to check the double precision floating tests on integer registers, which are expected to run on the current code changes (Through the PR is merged yet), may require a minor fixes on the framework with respect to "ISAC" and "CTG". Bit of a test driven changes we may have to carry ahead.
All the work related to Zdinx are completed with 100% coverage, now peer reviewing the reports. Also this has a dependency with the previous PR (zfh and zfinx) to be merged before raising a new PR for this work.
Coverage Report for RV32Zdinx are placed in the below link:
https://gitlab.com/ptprasanna/actreports/-/tree/main/RV32Zdinx?ref_type=heads
Coverage Reports for RV64Zdinx are placed in the below link:
https://gitlab.com/ptprasanna/actreports/-/tree/main/RV64Zdinx?ref_type=heads
Would you kindly provide a link to the PR(s) being reviewed for this? Nevermind....
@ptprasanna, I would like an update in the issue, by the next meeting on December 12, 2023, please.
All the work related to Zdinx are completed with 100% coverage reports. This has a dependency with the previous PR (zfh and zfinx).
Zfinx and Zfh PR for ISAC has been merged. https://github.com/riscv-software-src/riscv-isac/pull/72
Zfinx and Zfh PR from CTG has been waiting for review. https://github.com/riscv-software-src/riscv-ctg/pull/67
Previous PR(Zfinx and Zfh) to be merged before raising a new PR for this work.
@jjscheel,
Once Zfinx and Zfh PR from CTG has merged, I will raise a PR for Zdinx ACT.
Understand.
@allenjbaum, waiting here too.
@anuani21, I know that we are waiting for the PR in #17 and #18 to be accepted before building this PR. Given that we continue to wait for something, would it be worth our time to at least build this PR (assuming acceptance of the Zfinx and Zfh work) and submit for comment? Then, we could simply declare this "done" (or permanently blocked) and move on.
@jjscheel,
Updates from IITM,
I have a raised a PR for Zdinx extension
Thanks! I've updated the description above to include PRs.
@jjscheel,
Updates from IITM,
Re-submiited the PR in riscv-ctg and riscv-isac into riscv-arch-test https://github.com/riscv-non-isa/riscv-arch-test/pull/499
Technical Group
Architecture Test SIG
ratification-pkg
Zdinx
Technical Liaison
Allen Baum
Task Category
Arch Tests
Task Sub Category
Ratification Target
3Q2023
Statement of Work (SOW)
Component names:
Requirements: (prose describing what has to be done and what the end result is)
Deliverables:
Acceptance Criteria:
Projected timeframe: (best guess date)
SOW Signoffs: (delete those not needed)
Waiver
Pull Request Details