Skip to content

Conversation

@jackkoenig
Copy link
Contributor

@jackkoenig jackkoenig commented May 3, 2025

This debumps slightly to Verilator 5.020 2024-01-01 rev (Debian 5.020-1) (from currently build from source v5.022) but it seems to work and is simpler, especially as an example for users. What do you think @seldridge?

@jackkoenig jackkoenig requested a review from seldridge May 3, 2025 21:58
@tymcauley
Copy link

FWIW, I ran into some ChiselSim segfaults after the bump to firtool-1.115.0 while using Verilator 5.030, which were resolved by updating to Verilator 5.034.

@jackkoenig
Copy link
Contributor Author

FWIW, I ran into some ChiselSim segfaults after the bump to firtool-1.115.0 while using Verilator 5.030, which were resolved by updating to Verilator 5.034.

Good to know, thanks. Installing from source causes issues with giter81 due to $, but I will figure something out.

Footnotes

  1. experimenting with giter8 in https://github.com/jackkoenig/chisel-template.g8

Copy link
Member

@seldridge seldridge left a comment

Choose a reason for hiding this comment

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

This is probably fine.

There are (as Tynan states) some problems with older Verilator versions. Also, if you want force to work, you need 5.036 (which even the newest Ubuntu doesn't have).

Maybe what we want here is our own GitHub action just for installing Verilator. There are a couple on the Actions Marketplace already. However, this installation is so simple, that it may be easier (and safer) to roll it ourselves.

@jackkoenig jackkoenig force-pushed the jackkoenig/verilator branch from 9fe0b0f to 500696f Compare May 6, 2025 05:32
@jackkoenig jackkoenig force-pushed the jackkoenig/verilator branch from 500696f to 9d68b73 Compare May 6, 2025 05:32
@jackkoenig jackkoenig enabled auto-merge (squash) May 6, 2025 05:32
@jackkoenig jackkoenig merged commit e896c5c into main May 6, 2025
2 checks passed
@jackkoenig jackkoenig deleted the jackkoenig/verilator branch May 6, 2025 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants