pymtl / pymtl3

Pymtl 3 (Mamba), an open-source Python-based hardware generation, simulation, and verification framework
BSD 3-Clause "New" or "Revised" License
388 stars 46 forks source link

Verilator compile error #240

Closed hmohajeri closed 11 months ago

hmohajeri commented 1 year ago

This operating system is ubuntu-22 The verilator 4.036 no longer compiles under ubuntu. It looks like some problem with Bison header. The verilator version that do compile under ubuntu however give errors during link with pymtl3. Is there a solution to help this problem? Here is the sample output of one of the runs from the class

C compiler output: In file included from IntMulScycleV1_noparam_vALL_pickled.cpp:1: IntMulScycleV1_noparam_v.cpp: In function ‘void trace(VIntMulScycleV1_noparam_t, char)’: IntMulScycleV1_noparam_v.cpp:304:23: error: ‘class VIntMulScycleV1_noparam’ has no member named ‘VlSymsp’; did you mean ‘VIntMulScycleV1_noparam__Syms* const VIntMulScycleV1_noparam::vlSymsp’? (not accessible from this context) 304 svSetScope( &model->VlSymsp->Vscope_IntMulScycleV1_noparam__v ); ^~~~~ In file included from IntMulScycleV1_noparam_v.cpp:9, from IntMulScycleV1_noparam_v__ALL_pickled.cpp:1: obj_dir_IntMulScycleV1_noparam/VIntMulScycleV1_noparam.h:21:42: note: declared private here 21 VIntMulScycleV1_noparam__Syms* const vlSymsp;
cbatten commented 1 year ago

Hmmm ... no easy fix. @ptpan I think you had PyMTL3 working with the newest versions of Verilator right?

hmohajeri commented 1 year ago

Hi @ptpan , any feedback on this item?

ptpan commented 11 months ago

Hi @hmohajeri -- sorry for getting back to you so late.. I've been woking on adding support for a recent version of Verilator (v5.016). The support is now complete in our dev branch and we are planning for a major revision of pymtl (#237) in the near future so you can get that update from pypi. I apologize for this delay and hopefully this is helpful!

hmohajeri commented 11 months ago

No worries and many thanks, I do understand that you must be very busy and things take time.

all the best,

On Wed, Dec 6, 2023 at 4:05 PM Peitian Pan @.***> wrote:

Hi @hmohajeri https://github.com/hmohajeri -- sorry for getting back to you so late.. I've been woking on adding support for a recent version of Verilator (v5.016). The support is now complete in our dev branch and we are planning for a major revision of pymtl (#237 https://github.com/pymtl/pymtl3/pull/237) in the near future so you can get that update from pypi. I apologize for this delay and hopefully this is helpful!

— Reply to this email directly, view it on GitHub https://github.com/pymtl/pymtl3/issues/240#issuecomment-1843904370, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVO5E36G6X2BGB4BICOCXDYIEB6PAVCNFSM6AAAAAAURQSDRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBTHEYDIMZXGA . You are receiving this because you were mentioned.Message ID: @.***>