luke
/
verilator
mirror of https://github.com/verilator/verilator.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix `-j` ignored after `-f` (#5749).

This commit is contained in:
Wilson Snyder 2025-01-24 17:30:59 -05:00
parent d8080008b4
commit d4f4fefd1d
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Changes
View File

@ -36,6 +36,7 @@ Verilator 5.033 devel
* Fix `BLKSEQ` on suspendable processes (#5722). [Krzysztof Bieganski, Antmicro Ltd.] * Fix `BLKSEQ` on suspendable processes (#5722). [Krzysztof Bieganski, Antmicro Ltd.]
* Fix vpiDefName issues with non-inlined scopes and dpi conflicts (#5732). [Andrew Nolte] * Fix vpiDefName issues with non-inlined scopes and dpi conflicts (#5732). [Andrew Nolte]
* Fix duplicate-named class variable equivalence (#5737). * Fix duplicate-named class variable equivalence (#5737).
* Fix `-j` ignored after `-f` (#5749). [Luca Colagrande]
* Fix matching language extension options including dots. * Fix matching language extension options including dots.

11
src/V3Options.cpp
View File

@ -955,9 +955,6 @@ void V3Options::notify() VL_MT_DISABLED {
cmdfl->v3error("Unsupported: --timing and --savable not supported together"); cmdfl->v3error("Unsupported: --timing and --savable not supported together");
} }
// Mark options as available
m_available = true;
// --dump-tree-dot will turn on tree dumping. // --dump-tree-dot will turn on tree dumping.
if (!m_dumpLevel.count("tree") && m_dumpLevel.count("tree-dot")) { if (!m_dumpLevel.count("tree") && m_dumpLevel.count("tree-dot")) {
m_dumpLevel["tree"] = m_dumpLevel["tree-dot"]; m_dumpLevel["tree"] = m_dumpLevel["tree-dot"];
@ -965,9 +962,15 @@ void V3Options::notify() VL_MT_DISABLED {
// Sanity check of expected configuration // Sanity check of expected configuration
UASSERT(threads() >= 1, "'threads()' must return a value >= 1"); UASSERT(threads() >= 1, "'threads()' must return a value >= 1");
if (m_buildJobs == -1) m_buildJobs = 1;
if (m_verilateJobs == -1) m_verilateJobs = 1;
// Preprocessor defines based on options used // Preprocessor defines based on options used
if (timing().isSetTrue()) V3PreShell::defineCmdLine("VERILATOR_TIMING", "1"); if (timing().isSetTrue()) V3PreShell::defineCmdLine("VERILATOR_TIMING", "1");
// === Leave last
// Mark options as available
m_available = true;
} }
//###################################################################### //######################################################################
@ -1853,8 +1856,6 @@ void V3Options::parseOptsList(FileLine* fl, const string& optdir, int argc,
++i; ++i;
} }
} }
if (m_buildJobs == -1) m_buildJobs = 1;
if (m_verilateJobs == -1) m_verilateJobs = 1;
} }
//====================================================================== //======================================================================