Skip to content

Use Booth multipliers for GateMate #5200

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Use Booth multipliers for GateMate #5200

wants to merge 1 commit into from

Conversation

mmicko
Copy link
Member

@mmicko mmicko commented Jun 27, 2025

To cover soft logic implementation of multipliers using now Booth multipliers due to better resource usage. This makes some larger designs to place and route easier. This affects only leftovers or all multipliers in case -nomult is provided.

@pu-cc please check if you see any issues with this.

@pu-cc
Copy link
Contributor

pu-cc commented Jun 27, 2025

Thanks for letting me know, but I got some issues e.g. with vexriscv in litex. Without the “booth” pass, it basically always works without any problems, so I would prefer to test this on more examples first.

@povik
Copy link
Member

povik commented Jun 27, 2025

I got some issues e.g. with vexriscv in litex. Without the “booth” pass, it basically always works without any problems

Are those issues with place and route, or correctness issues?

@pu-cc
Copy link
Contributor

pu-cc commented Jun 27, 2025

Seems to be causing timing issues, so yes, place and route. But I am not yet working on @mmicko's improved_model2 branch in prjpeppercorn.

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

Successfully merging this pull request may close these issues.

3 participants