Make basedir overridable, add incdir optional harg #46
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
During development, it is convenient to integrate formal verification in the build process rather than integrating the core to the verification framework.
This PR removes the restriction to generate and run verification inside the riscv-formal dev tree
cores
directory:genchecks
now accepts an additional argument to overridebasedir
(a hard-coded reference to ../.. has been fixed too)genchecks
also provides a extra hargs:incdir
that can be passed as 3rd command-line argument to define additional include pathIn practice, this enables the following:
${ROOT_PROJECT}/platform/formal
containschecks.cfg
andwrapper.sv
checks.cfg
has a lineread_verilog -sv @incdir@/wrapper.sv
The following command will create a local, ready-to run directory structure:
NOTE:
genchecks
is assumingcfgname
is both the filename of the config file and the name of the target directory. This forces the config file to be copied in the build directory instead of just being referenced from the source directory. I couldn't find a way to do that in a backward-compatible way.