-
Notifications
You must be signed in to change notification settings - Fork 0
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
Zdinx ACT #19
Comments
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. Zfinx and Zfh PR from CTG has been waiting for review. Previous PR(Zfinx and Zfh) to be merged before raising a new PR for this work. |
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. |
Updates from IITM, I have a raised a PR for Zdinx extension |
Thanks! I've updated the description above to include PRs. |
Updates from IITM, Re-submiited the PR in riscv-ctg and riscv-isac into riscv-arch-test riscv-non-isa/riscv-arch-test#499 |
Zdinx PR is still under review. |
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
The text was updated successfully, but these errors were encountered: