YosysHQ / yosys

Yosys Open SYnthesis Suite
https://yosyshq.net/yosys/
ISC License
3.51k stars 895 forks source link

py_wrap: implement nested class definitions #4671

Closed widlarizer closed 1 month ago

widlarizer commented 1 month ago

Fixes #4670 by probably implementing nested class declaration. I don't really know what the wrapper generator does and I don't want to find out more than I already did. Also the DIY C++ parser doesn't like trailing commas in enum declarations so I removed it from RTLIL::ConstFlags.

povik commented 1 month ago

Is there a diff in the generator's output against what it produced the last time it worked?

widlarizer commented 1 month ago

Here you go. The diff is the const additions and then temporary variable naming noise

python_wrappersnew.txt python_wrappersold.txt

povik commented 1 month ago

The flags property seems to have disappeared

widlarizer commented 1 month ago

short -> short int fixes that

povik commented 1 month ago

Not the change I expected to help