CFU-Playground icon indicating copy to clipboard operation
CFU-Playground copied to clipboard

Add --no-std switch to verilator

Open PiotrZierhoffer opened this issue 3 years ago • 5 comments

@tcal-x can you please verify if this fixes the problem for you?

PiotrZierhoffer avatar Feb 24 '23 12:02 PiotrZierhoffer

Hi @PiotrZierhoffer , thanks! --- that did fix the issue I saw running "make renode" manually, but there are many failing tests now?

tcal-x avatar Feb 24 '23 18:02 tcal-x

It stems from the fact that the CI uses older verilator.

Let's [WIP] this - we'll change the package to implicitly provide this switch, so that the user interface doesn't change.

PiotrZierhoffer avatar Feb 27 '23 14:02 PiotrZierhoffer

If I recall correctly, we originally added Verilator to the list of Conda packages used because the version that we got from sudo apt install verilator in Google Colab (Ubuntu 18) was too old.

I thought all of the Github Actions now relied on the Conda packages, which should hide an older system version of Verilator?

tcal-x avatar Feb 27 '23 19:02 tcal-x

We made some changes to the verilator conda package, as it was failing the build on GCP.

We need to update it a little bit further to address this use case, I believe.

PiotrZierhoffer avatar Feb 28 '23 09:02 PiotrZierhoffer

@PiotrZierhoffer sorry if I dropped the ball here -- what is the status of this?

tcal-x avatar Apr 26 '23 17:04 tcal-x