Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] Improving Verilator model generation speed #1374

Closed
1 task done
pcotret opened this issue Sep 8, 2023 · 2 comments
Closed
1 task done

[BUG] Improving Verilator model generation speed #1374

pcotret opened this issue Sep 8, 2023 · 2 comments
Assignees
@JeanRochCoulon
Labels
Type:Item-of-task This issue is an item of a larger task.

Comments

@pcotret
Copy link
Contributor

pcotret commented Sep 8, 2023 •
edited
Loading

Is there an existing CVA6 bug for this?

  • I have searched the existing bug issues

Bug Description

In a recent PR, I've removed the -j(NUM_JOBS) because it crashed my virtual machine.

Unfortunately, generation of Verilator model is wayyyy slower...
Should I put it back in a new PR ? (I'll use NUM_JOBS=2 or 4 on my clone)
It may imply as well a short modification of the Readme when we perform make verilate ("you can use NUM_JOBS=$(nproc)" or something similar).
@pcotret pcotret added the Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system label Sep 8, 2023
@zarubaf zarubaf added Status:Stale Issue or PR is stale and hasn't received any updates. and removed Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system labels Oct 23, 2023
@github-actions github-actions bot removed the Status:Stale Issue or PR is stale and hasn't received any updates. label Oct 24, 2023
@JeanRochCoulon
Copy link
Contributor

Related to #1569

@JeanRochCoulon JeanRochCoulon added the Type:Item-of-task This issue is an item of a larger task. label Oct 24, 2023
@zchamski
Copy link
Contributor

One possible improvement is to remove the -O3 option passed to Verilator. According to VL documentation it mainly applies inlining and this may significantly slow down C++ compilation.

@pcotret pcotret closed this as completed Nov 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JeanRochCoulon JeanRochCoulon

Labels
Type:Item-of-task This issue is an item of a larger task.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@zarubaf @pcotret @JeanRochCoulon @zchamski