-
Notifications
You must be signed in to change notification settings - Fork 877
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
ABC: execution of command "..." failed: return code 134. #1675
Comments
Full build log of working commit: (Build 222) https://travis-ci.com/adumont/hrm-cpu/builds/147103961#L1596 Build log of failing commit: (Build 223) https://travis-ci.com/adumont/hrm-cpu/builds/147104799#L1482 |
Hi @adumont. Thanks for the report. Your diff looks innocuous enough that this looks to be a bug in upstream ABC berkeley-abc/abc, perhaps hitting an obscure corner case that wasn't triggered before. Would you like to report it there? An example that I did earlier is here: berkeley-abc/abc#61 and uses the equivalent test-case that you attached as
As for you current predicament, I understand that the Let me know if that unblocks you! |
Hi @eddiehung , Thanks for the feedback. First of all, I have tried the I'll open an issue in berkeley-abc/abc. Feel free to close this issue then :). Thanks a lot. |
The assertion is commented out for now in abc: berkeley-abc/abc@53f2238 |
Thanks @nakengelhardt! I think that change is safe to use despite its commit message being:
I've opened PR #1744 to bump |
Bump ABCREV to receive fix for #1675
Because YosysHQ/yosys#1675 got fixed
Because YosysHQ/yosys#1675 got fixed
Steps to reproduce the issue
Failing commit is 2b4180c2bf508fead7d1d14def8600957124f50b.
will try to synthesize the design. The step:
fails with:
I've uploaded some stuff from the /tmp/yosys-abc-eQGZWS:
yosys-abc-eQGZWS.zip
My toolchain is updated to latest git version (tool/commit sha):
yosys.ver:9f5613100b360beb60608df1296ee81dc185e56c
arachne-pnr.ver:c40fb2289952f4f120cc10a5a4c82a6fb88442dc
icestorm.ver:0ec00d892a91cc68e45479b46161f649caea2933
iverilog.ver:b1114760fcfba55ee9ca35ba95e2b3ed2b7c7a20
symbiyosys.ver:500b526131f434b9679732fc89515dbed67c8d7d
verilator.ver:0c6c83e2787803d9b7c014ddbc8710f47d0f82f7
yices2.ver:1dcb71c405b5f1d21b363eb33b566068cad2e87d
Previous commit (053c6f1a5f2ab79add807138b24f92de6b3b308c053c6f1a5f2ab79add807138b24f92de6b3b308c) build fine.
Here's the diff in verilog code: adumont/hrm-cpu@2b4180c.
Expected behavior
It should produce the blif file.
Actual behavior
Fails with assertion in ABC.
The text was updated successfully, but these errors were encountered: