Skip to content

Conversation

seldridge
Copy link
Member

@seldridge seldridge commented Oct 8, 2025

Add Verilator compilation-time options to the Verilator svsim backend.
This is added to workaround an issue observed internally where Verilator
can crash when running with full parallelism, i.e., the default -j 0
that was previously hard-coded in the Verilator backend.

The default behavior is preserved, but users can now set -j OR
--build-jobs/--verilate-jobs individually.

Release Notes

Add control of Verilator build parallelism to svsim.

Add Verilator compilation-time options to the Verilator svsim backend.
This is added to workaround an issue observed internally where Verilator
can crash when running with full parallelism, i.e., the default `-j 0`
that was previously hard-coded in the Verilator backend.

The default behavior is preserved, but users can now set `-j` OR
`--build-jobs`/`--verilate-jobs` individually.

Signed-off-by: Schuyler Eldridge <[email protected]>
@seldridge seldridge requested a review from jackkoenig October 8, 2025 15:59
@seldridge seldridge added the Feature New feature, will be included in release notes label Oct 9, 2025
@seldridge seldridge force-pushed the dev/seldridge/svsim-expose-verilator-compile-time-parallelism-options branch from 4992a93 to 24ee0e1 Compare October 9, 2025 05:00
Copy link
Contributor

@jackkoenig jackkoenig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, see comments, consider making new CompilationSettings constructor private

@seldridge seldridge force-pushed the dev/seldridge/svsim-expose-verilator-compile-time-parallelism-options branch from 77d26fa to 6decefd Compare October 10, 2025 04:08
@seldridge seldridge enabled auto-merge (squash) October 10, 2025 05:30
@seldridge seldridge merged commit 78ba535 into main Oct 10, 2025
16 checks passed
@seldridge seldridge deleted the dev/seldridge/svsim-expose-verilator-compile-time-parallelism-options branch October 10, 2025 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature, will be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants