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

verilog: add memref.load, func.call and arithmetic index ops support to emitter #681

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

Conversation

copybara-service[bot]
Copy link
Contributor

@copybara-service copybara-service bot commented May 13, 2024

verilog: add memref.load, func.call and arithmetic index ops support to emitter

This adds support for function calls in the generic body as separate verilog submodules, and support for variable index load/stores (so we can support tosa::table ops!).

Note: this also updates yosys-optimizer to handle the function calls. I found out that splitnets -ports generic_body %n can split the output ports of all BUT generic_body (that's what %n does to a selection).

@copybara-service copybara-service bot force-pushed the test_633240305 branch 2 times, most recently from 10645d2 to 817cfdc Compare May 13, 2024 17:34
@copybara-service copybara-service bot changed the title feat: add memref.load, func.call and arithmetic index op support to verilog emitter verilog: add memref.load, func.call and arithmetic index ops support to emitter May 13, 2024
…to emitter

This adds support for function calls in the generic body as separate verilog submodules, and support for variable index load/stores (so we can support tosa::table ops!).

Note: this also updates yosys-optimizer to handle the function calls. I found out that `splitnets -ports generic_body %n` can split the output ports of all BUT generic_body (that's what %n does to a selection).
PiperOrigin-RevId: 633240305
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.

None yet

1 participant