#56 create_testsuite.pl should support "module automatic"

[Victor Lyuboslavsky]

Verilog modules can have a lifetime specifier, like:

module automatic svunitDemo_unit_test;

or

module static svunitDemo_unit_test;

This is currently not supported by create_testsuite.pl due to this:

if ( /^\s*module\s*(\w+_unit_test);/ ) {
  push (@unittests, $1);
}

Issue example (note that no testcases ran): http://www.edaplayground.com/x/WJ

In fact, I think using automatic storage for unit test modules should be an SVUnit recommendation. This will use automatic storage for any functions/tasks inside the module.

[Victor Lyuboslavsky]

Luckily, there is a funny workaround. See: http://www.edaplayground.com/x/Xu

// Funny workaround for https://sourceforge.net/p/svunit/tickets/56/
`define WORKAROUND_svunitDemo_unit_test \
module svunitDemo_unit_test;

module automatic svunitDemo_unit_test;

[Neil Johnson]

fixed in frmwrk_20 unit test. released in 2.8.

[Victor Lyuboslavsky]

This doesn't fix the issue for me. I see that automatic is now being considered in create_unit_test.pl, but create_testsuite.pl still does not support it.

My flow is the following:

$SVUNIT_INSTALL/bin/create_testsuite.pl -add ./testbench.sv -out _testsuite.sv && $SVUNIT_INSTALL/bin/create_testrunner.pl -add ./_testsuite.sv -out _testrunner.sv && <run sim>

Perhaps I should be using a different way to create testsuite/testrunner files?

[Neil Johnson]

I completely missed the point here but I think I've got it fixed up now. released in v2.10.