cva6 icon indicating copy to clipboard operation
cva6 copied to clipboard

[TASK] Add coverage dedicated to Zce to ISACOV

Open JeanRochCoulon opened this issue 2 years ago • 8 comments

What (objective description) ? Add covergroups for new extension Zicond and Zca, Zcb, Zcmp, Zcmt (Zce for RV32 without F).

How to ? I prefer to add these covergroup in the cva6 env, because it dediceted to extension related only for embedded cva6.

Current Status ? None.

Risks ? None

Prerequisites ? Zca, Zcb, Zcmp, Zcmt spec

KPI ? None

Description of done? Available Function coverage for new extension.

JeanRochCoulon avatar Sep 13 '23 13:09 JeanRochCoulon

Any news ? If some PRs have been merged, please refer the PR number.

JeanRochCoulon avatar Oct 10 '23 04:10 JeanRochCoulon

I add the coverage for Zicond (# 1473) For Zce is pending (do you mean Zcb ?)

AyoubJalali avatar Oct 10 '23 14:10 AyoubJalali

Zicond has been disabled in embedded configuration #1534

JeanRochCoulon avatar Oct 17 '23 15:10 JeanRochCoulon

Waiting for the merge of the ISA_DECODER into master, to use it to decode these new instruction, because the dasm spike does not support them.

AyoubJalali avatar Oct 26 '23 12:10 AyoubJalali

HEllo @AyoubJalali What is the status ? Can we consider it as completed ?

JeanRochCoulon avatar May 22 '24 08:05 JeanRochCoulon

For now there's only coverage for Zcb !!

AyoubJalali avatar May 22 '24 08:05 AyoubJalali

For now the CV32A65X does not support ZCMP or ZCMT, should I add covergoup dedicated for them ? @JeanRochCoulon

AyoubJalali avatar Nov 05 '24 12:11 AyoubJalali

65X will support them. I do not know whether you need to have the design implemntation to test your covergroup. You can decide when implementing them.

JeanRochCoulon avatar Nov 05 '24 16:11 JeanRochCoulon