Skip to content

[NOT MERGE] add coverage option to collect line/toggle coverage #932

Draft
Clo91eaf wants to merge 4 commits intomasterfrom
Clo91eaf/other-coverage-option
Draft

[NOT MERGE] add coverage option to collect line/toggle coverage #932
Clo91eaf wants to merge 4 commits intomasterfrom
Clo91eaf/other-coverage-option

Conversation

@Clo91eaf
Copy link
Copy Markdown
Contributor

@Clo91eaf Clo91eaf commented Jan 8, 2025

This pr implements line/toggle or any other coverage type in the cover flow of t1. To achieve this, I added the vcs-emu-cover-full target in nix.

  • Compile time:

Use -cm line+cond+fsm+tgl+branch+assert to enable all possible coverage types. This change may greatly affect the ci running time, so this PR will NOT BE MERGE.

  • Runtime:
  1. Run the regression test in ci: At this time, only the coverage of line+tgl+assert is collected. This process is hard-coded and can be changed by modifying the nix script.
    After the ci operation is completed, you can obtain all the coverage through this command.
git checkout Clo91eaf/other-coverage-option
nix build .#t1.blastoise.t1rocketemu.run._vcsEmuResult --impure
urg -dir ./result/*/cm.vdb -full64 -dbname merged -show summary -parallel
verdi -cov -covdir merged.vdb
  1. To run a single test case locally: you can specify the test type you want to run through the cover-type parameter of t1-helper, for example:
nix develop -c t1-helper run -i t1rocketemu -e vcs-emu-cover-full -c blastoise --cover-type line+tgl+assert codegen.vsra_vx

@Clo91eaf Clo91eaf changed the title Clo91eaf/other coverage option add coverage option to collect line/toggle coverage Jan 8, 2025
@Clo91eaf Clo91eaf force-pushed the Clo91eaf/other-coverage-option branch from 37cc516 to 531f345 Compare January 9, 2025 02:07
@Clo91eaf Clo91eaf changed the title add coverage option to collect line/toggle coverage [NOT MERGE] add coverage option to collect line/toggle coverage Jan 9, 2025
@Clo91eaf Clo91eaf assigned Clo91eaf and unassigned Clo91eaf Jan 9, 2025
@Clo91eaf Clo91eaf force-pushed the Clo91eaf/other-coverage-option branch 16 times, most recently from 4dd9c1c to 8813d79 Compare January 10, 2025 04:15
@Clo91eaf Clo91eaf force-pushed the Clo91eaf/other-coverage-option branch from c0d6df1 to 2b3e595 Compare January 16, 2025 14:32
Signed-off-by: Clo91eaf <Clo91eaf@qq.com>
Signed-off-by: Clo91eaf <Clo91eaf@qq.com>

[ci] run urg parallelly

Signed-off-by: Clo91eaf <Clo91eaf@qq.com>
@Clo91eaf Clo91eaf force-pushed the Clo91eaf/other-coverage-option branch from 2b3e595 to 21fc171 Compare January 16, 2025 17:18
Signed-off-by: Clo91eaf <Clo91eaf@qq.com>
@Clo91eaf Clo91eaf force-pushed the Clo91eaf/other-coverage-option branch from 21fc171 to 9546c44 Compare January 17, 2025 05:31
Signed-off-by: Clo91eaf <Clo91eaf@qq.com>
@sequencer sequencer marked this pull request as draft April 27, 2025 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant