search

lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.53k stars 754 forks source link

[verilator] make verilator sim binary buildable with foundry repo present #15882

Open timothytrippel opened 1 year ago

timothytrippel commented 1 year ago

At them moment, the Verilator simulation binary fails to build if the foundry repo is present.

Long term this should be fixed, e.g., by #ifdef-ing HDL blocks away that cause build failures), but in the short term, we can tell bazel to ignore files in hw/foundry.

timothytrippel commented 1 year ago

The short term fix has been merged in #15885, but keep this open until the long term fix is complete.

drewmacrae commented 1 year ago

I feel like ignoring the files is going to be more performant if we can get away with it. We don't have to set verilator loose on all files. We could rename the package and include only the necessary subset.