Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Compiler integration of the Concat operation #1193

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

mdanilow
Copy link

@mdanilow mdanilow commented Sep 20, 2024

This PR depends on a PR from the finn-hlslib where the Concat operation was implemented (Xilinx/finn-hlslib#143)

  • The Concat op code is not hardcoded in the compiler anymore and it now accepts different datatypes of inputs. The output datatype is inferred.
  • HLS implementation is defined for a one clock cycle, without defining NUM_REPS, so the new timeout cppsim template is also added and utilized by the operation.
  • The tests/fpgadataflow/test_fpgadataflow_concat.py test now tests the case of inputs being different datatypes

Michal Danilowicz added 9 commits September 20, 2024 16:12
…eNHWC transformations are checking whether the node to be moved is a fork node, in which case the MoveOpPastFork is called. MoveOpPastFork uses deepcopies of the original node.
… and it now accepts different datatypes of inputs. It uses the new implementation from finn-hlslib
…ypes among inputs and sets the SIMD parameter
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant