Skip to content

Add check for existing coverage files to merge_coverage function (mod…#1210

Open
patrick-studer wants to merge 1 commit into
VUnit:masterfrom
patrick-studer:fix-merge-coverage
Open

Add check for existing coverage files to merge_coverage function (mod…#1210
patrick-studer wants to merge 1 commit into
VUnit:masterfrom
patrick-studer:fix-merge-coverage

Conversation

@patrick-studer

@patrick-studer patrick-studer commented Jul 15, 2026

Copy link
Copy Markdown

If simulation is run with coverage enabled but no testcases were executed (because filtering), the merge_coverage procedure does end in an error. This pre-check checks, if there is anything to merge before performing the merge command.

Resolves #1208.

Result (with ModelSim):

$ make sim
WARNING - Found no test benches using current filter rule:

    Filters entities and modules that have a runner_cfg generic/parameter

    Gives warning when design_unit matches tb_* or *_tb without having a runner_cfg
    Gives warning when a design_unit has  a runner_cfg but the name does not match tb_* or *_tb

Re-compile not needed

No tests were run!
WARNING - No existing coverage files found. Skipping coverage database creation.

…elsim)

If simulation is run with coverage enabled but no testcases were executed (because filtering), the merge_coverage procedure does end in an error. This pre-check checks, if there is anything to merge before performing the merge command.
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.

Missing check for existing coverage files on merge_coverage function (modelsim)

1 participant