Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Performance regression in Yosys 0.39 #4280

Open
whitequark opened this issue Mar 13, 2024 · 6 comments
Open

Performance regression in Yosys 0.39 #4280

whitequark opened this issue Mar 13, 2024 · 6 comments
Labels
pending-verification This issue is pending verification and/or reproduction

Comments

@whitequark
Copy link
Member

whitequark commented Mar 13, 2024 •
edited
Loading

Version

0.39

On which OS did this happen?

Linux [I think this needs a "WebAssembly" choice; #4281]

Reproduction Steps

Run Glasgow Interface Explorer testcase glasgow.applet.interface.analyzer.test.AnalyzerAppletTestCase.test_build

Expected Behavior

Build passes timing.

Actual Behavior

Build does not pass timing since Yosys 0.39.
@whitequark whitequark added the pending-verification This issue is pending verification and/or reproduction label Mar 13, 2024
@whitequark whitequark mentioned this issue Mar 13, 2024
Merged
@povik
Copy link
Member

povik commented Mar 13, 2024

@whitequark Do we know v0.38 is fine?

@whitequark
Copy link
Member Author

Yes, here is an equivalent build using Yosys 0.38+0

@whitequark
Copy link
Member Author

I actually can't see any commit that should influence this, so maybe it was just a marginal test that worked for like a dozen or two Yosys versions but started failing now?

@povik
Copy link
Member

povik commented Mar 13, 2024

I wonder too. I guess you can ask github to rerun the test, to see if it fails reliably?

@povik
Copy link
Member

povik commented Mar 13, 2024

I guess you can ask github to rerun the test, to see if it fails reliably?

Ah, but it probably does fail reliably, because the synthesis is deterministic though chaotic. That's what you are getting at when you say it worked for a couple of Yosys versions...

@whitequark
Copy link
Member Author

Yes.

whitequark added a commit to whitequark/glasgow that referenced this issue Mar 14, 2024
@whitequark
applet.interface.analyzer: restrict to revC0+.
04c1526 
After upgrading Yosys to 0.39, the applet stopped passing timing when
built for revA. This was reported against Yosys (YosysHQ/yosys#4280),
but as there don't seem to be any commits that could change synthesis
quality, it was likely just that the analyzer applet was marginal in
first place.

Since this is blocking all further development, the applet is now
restricted to revC0+.
@whitequark whitequark mentioned this issue Mar 14, 2024
Merged
github-merge-queue bot pushed a commit to GlasgowEmbedded/glasgow that referenced this issue Mar 14, 2024
@whitequark
applet.interface.analyzer: restrict to revC0+.
6f20482 
After upgrading Yosys to 0.39, the applet stopped passing timing when
built for revA. This was reported against Yosys (YosysHQ/yosys#4280),
but as there don't seem to be any commits that could change synthesis
quality, it was likely just that the analyzer applet was marginal in
first place.

Since this is blocking all further development, the applet is now
restricted to revC0+.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pending-verification This issue is pending verification and/or reproduction
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@whitequark @povik