Closed dapfister313 closed 4 months ago
Thanks for the contribution! I haven't had the time to look at it beyond a very far away glance (and will most likely not have time before the beginning of March unfortunately, having important deadlines on my side), but I already have a couple comments.
clang-tidy
generates. On VSCode both clang-format
and clang-tidy
are provided by the llvm-vs-code-extensions.vscode-clangd
extension.main
as GitHub indicates, this needs to be fixed.For reference, all of these recommendations are in our contribution guidelines.
I'm not entirely sure, but I think, I resolved the conflicts.
This PR introduces the ResourceSharingFCCM22Pass. This implements the methodology described in paper "Resource Sharing of Dataflow Circuits" presented at FCCM '22. There is the implementation of the complete algorithm that finds a grouping scheme for operations of the same type, such that there is no performance degradation when a group of operations share the same physical functional unit.
For example, the following outputs the best possible grouping scheme for the multipliers: