-
Notifications
You must be signed in to change notification settings - Fork 10
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
Improve arithmetic #5
Comments
I hope there is a way Yosys can be smarter about using the adder for other things. Multiplication also generates huge chunks of logic without a single adder. It'd probably be possible to implement multiplication ourselves in a techpass, but it'd be worth looking into letting Yosys do the heavy lifting. |
I'm trying to debug an issue where everything gets optimized away if I use a multiplication in one particular case. Meanwhile I found this: https://github.com/YosysHQ/yosys/blob/master/techlibs/common/techmap.v#L260 so if we make a $lcu from 74182 it would get used already. But actually it's not like I can find 74182 for sale anywhere. Hmmm, I wonder why big commercial companies stopped producing obsolete logic chips... |
Fun fact: |
Just a comment with random things I want to look at later so I don't forget:
|
Yosys is not happy with my J-K flip-flop. Probably not worth spending more time on. Was just a curiosity:
It was fine with me putting an 8-input NAND in, but never used it at all, even when writing
|
Currently we only generate $add cells based on the 74AC283, but Yosys has a bunch of other things that we could look into.
Summary of the discussion so far as I understand it:
The text was updated successfully, but these errors were encountered: