KULeuven-MICAS / snax_cluster

A heterogeneous accelerator-centric compute cluster
Apache License 2.0
10 stars 9 forks source link

New Cut in Chisel #342

Closed IveanEx closed 1 month ago

IveanEx commented 1 month ago

This PR aims to work with @xiaoling-yi to investigate and fix the long critical path problem exists in our circuit, so that we can goes to high frequency.

To fulfill this target, this PR implements a new DataCut component, which datapath is implemented by shift registers and Controller controls the behavior of the system to compliant with Valid / Ready signal handshaking protocol.

To ease the usage, the new operator is introduced to insert it: -\>, -\\> and -\\\>.

Please be aware that the Control Signal is not cut. To also cut the Control signal, you should use -|> or -||> combined with -\>, -\\> and -\\\>. The former one (-|> or -||>) is not cannot be retimed, because it is implemented by Ping-Pong architecture.

rgantonio commented 1 month ago

PR description, please! 😄

IveanEx commented 1 month ago

PR description, please! 😄

Available now!

rgantonio commented 1 month ago

That's a neat trick! Okay go go go!