-
Notifications
You must be signed in to change notification settings - Fork 544
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
False else points 4691 #4844
base: master
Are you sure you want to change the base?
False else points 4691 #4844
Conversation
…or into false_else_points_4691
I added --main support for coverage so you shouldn't need a .cpp file any longer, see the t_cover_main.pl test for an example. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should now be able to delete test_regress/t/t_cover_else_points.cpp.
top_filename("t/t_cover_else_points.sv"); | ||
golden_filename("t/t_cover_else_points.out"); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
top_filename("t/t_cover_else_points.sv"); | |
golden_filename("t/t_cover_else_points.out"); |
use the defaults and rename your file to .v instead of .sv. Then this test passes for me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ubuntu 20.04 gcc tests are still failing despite these changes.
# Version 2.0. | ||
# SPDX-License-Identifier: LGPL-3.0-only OR Artistic-2.0 | ||
|
||
scenarios(simulator => 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
scenarios(simulator => 1); | |
scenarios(vlt => 1); |
"$Self->{obj_dir}/coverage.dat", | ||
], | ||
verilator_run => 1, | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
); | |
) if !$self->errors && !$self->skips; |
As Ubuntu 20 is failing due to a skip from needing coroutines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That does not seem to work locally. I am getting the following error.
- t/t_cover_else_points.v:59: Verilog $finish
Can't call method "errors" on an undefined value at t/t_cover_else_points.pl line 25.
Compilation failed in require at ./driver.pl line 842.
Line 25 is the beginning for the run
command.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My fault, should be Self, capitalized.
Created a test case to replicate issue #4691