search

google / CFU-Playground

Want a faster ML processor? Do it yourself! -- A framework for playing with custom opcodes to accelerate TensorFlow Lite for Microcontrollers (TFLM). . . . . . Online tutorial: https://google.github.io/CFU-Playground/ For reference docs, see the link below.
http://cfu-playground.rtfd.io/
Apache License 2.0
452 stars 116 forks source link

Add --no-std switch to verilator #783

Open PiotrZierhoffer opened 1 year ago

PiotrZierhoffer commented 1 year ago

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

tcal-x commented 1 year ago

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

PiotrZierhoffer commented 1 year ago

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.

tcal-x commented 1 year ago

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?

PiotrZierhoffer commented 1 year ago

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.

tcal-x commented 1 year ago

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