Skip to content

Migrate yosys and ABC to bazelmod#2555

Merged
copybara-service[bot] merged 1 commit intogoogle:mainfrom
j2kun:yosys-bzlmod
Feb 24, 2026
Merged

Migrate yosys and ABC to bazelmod#2555
copybara-service[bot] merged 1 commit intogoogle:mainfrom
j2kun:yosys-bzlmod

Conversation

@j2kun
Copy link
Copy Markdown
Collaborator

@j2kun j2kun commented Jan 21, 2026

Fixes #1446

Depends on bazelbuild/bazel-central-registry#7258
Rebased over #2548

This will require some nontrivial internal patches for copybara

@j2kun j2kun requested a review from asraa January 21, 2026 23:29
@j2kun
Copy link
Copy Markdown
Collaborator Author

j2kun commented Feb 4, 2026

OK, one more build error, looks like the external kernel module is not pulling in all the cpp files it needs

ld.lld: error: undefined reference: Yosys::RTLIL_BACKEND::dump_sigspec(std::ostream&, Yosys::RTLIL::SigSpec const&, bool)
>>> referenced by bazel-out/k8-opt/bin/_solib_k8/libexternal_Syosys+_Slibkernel.so (disallowed by --no-allow-shlib-undefined)

@j2kun
Copy link
Copy Markdown
Collaborator Author

j2kun commented Feb 10, 2026

Update: I got it building and tests passing locally after bazelbuild/bazel-central-registry#7507

@j2kun j2kun force-pushed the yosys-bzlmod branch 5 times, most recently from 47e97d9 to a0839ec Compare February 20, 2026 17:36
Note this required removing the functional testing (run_verilog)
of emitted verilog because it appears that was removed from upstream
yosys. I cannot find the `eval` command anymore. If that is restored,
and exposed in the upstream bazel module, then we can restore those
tests.
@j2kun j2kun requested a review from asraa February 20, 2026 17:54
@j2kun
Copy link
Copy Markdown
Collaborator Author

j2kun commented Feb 20, 2026

OK it's ready!

@j2kun j2kun added the pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing label Feb 23, 2026
@copybara-service copybara-service Bot merged commit 5d7e781 into google:main Feb 24, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pull_ready Indicates whether a PR is ready to pull. The copybara worker will import for internal testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate remaining dependencies to Bazelmod

2 participants