Skip to content

yosys_update -> v0.38 #2616

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

yosys_update -> v0.38 #2616

wants to merge 1 commit into from

Conversation

amirarjmand93
Copy link
Contributor

Description

This update brings Yosys to version 0.38, which includes several bug fixes, performance improvements, and new features that enhance the synthesis capabilities of our flow. Notable additions in this version include :
-New commands and options

For a detailed list of changes and improvements in Yosys v0.38 (Feb 9, 2024), please refer to the release notes.

Related Issue

-Update Yosys [The current version in the VTR flow is 0.32]

Types of changes

  • New feature (change which adds functionality)
  • I have added tests to cover my changes
  • All new and existing tests passed

@github-actions github-actions bot added lang-cpp C/C++ code lang-python Python code lang-make CMake/Make code lang-hdl Hardware Description Language (Verilog/VHDL) lang-shell Shell scripts (bash etc.) Parmys labels Jun 14, 2024
@vaughnbetz
Copy link
Contributor

Thanks @amirarjmand93 . I checked one of the CI failures and it is OK (it's a logic reduction on a small design):
https://github.com/verilog-to-routing/vtr-verilog-to-routing/actions/runs/9510083459/job/26214028541?pr=2616

Hopefully the rest are like that and can be handled by updating some golden results.

@amirarjmand93
Copy link
Contributor Author

Result
Overall, in comparison to Yosys v0.42, we have less violent failure in the QoR test in Yosys v0.38.
But outdated f4pga still remains as the main external problem.

Primary Errors:

Conflict from f4pga to yosys v0.38 .

  • f4pga looking for 'process_format_str' in Yosys . like 'multirange_dimensions' in Yosys v0.42, this item also has been changed in recent versions. (funcrion name/functionality/output have been changed in new simplify.cc)

File path(after making):
vtr-verilog-to-routing/libs/EXTERNAL/yosys-f4pga-plugins/systemverilog-plugin/third_party/yosys/simplify.cc

Command:
make -k CMAKE_PARAMS=-DYOSYS_F4PGA_PLUGINS=ON -j16

Logs:
root/vtr-verilog-to-routing/vtr-verilog-to-routing/libs/EXTERNAL/yosys-f4pga-plugins/systemverilog-plugin/third_party/yosys/simplify.cc: In function 'bool systemverilog_plugin::simplify(Yosys::AST::AstNode*, bool, bool, bool, int, int, bool, bool)':
2024-06-14T03:33:58.3180898Z �[32;1m03:33:58�[0m | /root/vtr-verilog-to-routing/vtr-verilog-to-routing/libs/EXTERNAL/yosys-f4pga-plugins/systemverilog-plugin/third_party/yosys/simplify.cc:728:32: error: 'struct Yosys::AST::AstNode' has no member named 'process_format_str'; did you mean 'processFormat'?

Related issue:
Yosys Update -> v0.42
SystemVerilog support for Yosys -> CI failed

I think we should turn the f4pga off in the CI test till that regains compatibility with Yosys.

@vaughnbetz
Copy link
Contributor

Closing, as we upgraded all the way to 0.42 instead.

@vaughnbetz vaughnbetz closed this Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-cpp C/C++ code lang-hdl Hardware Description Language (Verilog/VHDL) lang-make CMake/Make code lang-python Python code lang-shell Shell scripts (bash etc.) Parmys
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants