-
Notifications
You must be signed in to change notification settings - Fork 52
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
Initial covergroups for Zvk* instructions #61
Conversation
This patch introduces initial versions of covergroups for the Zvk* instructions. The lack of vector support makes these pretty useless, but once this is available it should not be too hard to adjust these. Support for RV32 and RV64 is added, but both versions are identical. Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
Signed-off-by: Umer Shahid <umer.shahid@10xengineers.ai>
Updated ChangeLOG file and resolved conflicts Signed-off-by: Umer Shahid <umer.shahid@10xengineers.ai>
Since it was a very old PR, I have resolved its conflicts |
Merged version number to resolve the conflict with __init__.py Signed-off-by: Umer Shahid <umer.shahid@10xengineers.ai>
There are no red flags in the review, we can merge it but it won't be useful for us as vector tests cant be generated through ctg so there is no point in keeping vector registers in here. |
These coverpoints are incomplete and also outside the scope of this repo as vector tests will not be generated through ctg. I am closing this one to clear the backlog. |
We are letting this one and riscv-arch-tests (PR#33) go into the dev as riscv-arch-tests is the right platform where vector/crypto tests will be collected eventually. |
This patch introduces initial versions of covergroups for the Zvk* instructions.
The lack of vector support makes these pretty useless, but once this is available it should not be too hard to adjust these.
Support for RV32 and RV64 is added, but both versions are identical.