search

YosysHQ / yosys

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

Performance regression in Yosys 0.39 #4280

Open whitequark opened 3 months ago

whitequark commented 3 months ago

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.

povik commented 3 months ago

@whitequark Do we know v0.38 is fine?

whitequark commented 3 months ago

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

whitequark commented 3 months ago

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 commented 3 months ago

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

povik commented 3 months ago

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 commented 3 months ago

Yes.