Skip to content

Conversation

@Zxis233
Copy link

@Zxis233 Zxis233 commented Jan 18, 2026

Fix the prim module path and the kGE area calculation to adapt Yosys 0.60

Copy link
Contributor

@marnovandermaas marnovandermaas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you share the command you used to test this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you explain the changes in this file. Previously the code used data 0 and data 1. Now you are also using data 2 but not data 1.

Copy link
Author

@Zxis233 Zxis233 Jan 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you explain the changes in this file. Previously the code used data 0 and data 1. Now you are also using data 2 but not data 1.

The area report in Yosys 0.60 contains content similar to the following:

        +----------Local Count, excluding submodules.
        |        +-Local Area, excluding submodules.
        |        | 
      372  395.808   AND2_X1
       98   130.34   AND3_X1

That is, the real cell name is the third field, while the old code assumed that the cell name is the first field

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation. Is that a change that was introduced in Yosys?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation. Is that a change that was introduced in Yosys?

I've checked the commit history and I find it:
https://github.com/YosysHQ/yosys/commit/a0dde68487da600c569c0b61b684dc845fd5c554

Clearly the area report had been refactored, but it was NOT mentioned in the update log.

@Zxis233
Copy link
Author

Zxis233 commented Jan 20, 2026

Can you share the command you used to test this?

As the Wiki shows:

cd syn && ./syn_yosys.sh

@marnovandermaas marnovandermaas changed the title Fix the prim module path [syn] Adapt for Yosys 0.60 Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants