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.

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

xaiger failure during abc9 pass #2343

Closed
gsomlo opened this issue Aug 16, 2020 · 2 comments
Closed

xaiger failure during abc9 pass #2343

gsomlo opened this issue Aug 16, 2020 · 2 comments
Labels

Comments

@gsomlo
Copy link
Contributor

gsomlo commented Aug 16, 2020

Steps to reproduce the issue

See gls_mvce.tar.xz. Reproduce with:

tar xfJ gls_mvce.tar.xz
cd gls_mvce
bash RunMe.sh

Expected behavior

yosys successfully completes synthesis run, including abc9 techmapping and aiger stage

Actual behavior

yosys fails with:

8.47.18.6. Executing AIGER frontend.
/usr/include/c++/10/bits/stl_vector.h:1045: std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp, _Alloc>::size_type) [with _Tp = Yosys::hashlib::dict<Yosys::RTLIL::SigBit, Yosys::RTLIL::State>::entry_t; _Alloc = std::allocator<Yosys::hashlib::dict<Yosys::RTLIL::SigBit, Yosys::RTLIL::State>::entry_t>; std::vector<_Tp, _Alloc>::reference = Yosys::hashlib::dict<Yosys::RTLIL::SigBit, Yosys::RTLIL::State>::entry_t&; std::vector<_Tp, _Alloc>::size_type = long unsigned int]: Assertion '__builtin_expect(__n < this->size(), true)' failed.

Reproducer tarball includes a backtrace in yosys-abc-QNqgGU-backtrace.txt and the entire yosys-abc temp folder yosys-abc-QNqgGU.

I am using upstream abc at commit 448f263 with PR 89 applied on top (which gets rid of a problem whereby abc itself crashes during the abc9 techmapping stage, see issue 84. Given the recent commit version of abc I'm using, I believe these problems are also shared with the current "vendored" abc fork maintained by yosyshq.

For context, the verilog files in the reproducer are building a LiteX + RocketChip SoC (edit: for the ecp5-versa-5g board from Lattice).

Happy to provide any extra details to help further narrow down the problem. Thanks!

@Ravenslofty
Copy link
Collaborator

cc @eddiehung

@whitequark whitequark added the bug label Dec 5, 2020
@eddiehung
Copy link
Collaborator

No longer reproduces on vanilla b2e9717 -- the attached testcase gets all the way through the flow. Closing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants