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

Add repeat support for compute tiles #1842

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Conversation

AndraBisca
Copy link
Collaborator

@AndraBisca AndraBisca commented Oct 16, 2024

Currently, a repeat value can be specified for Mem tiles via the objectfifo's memtile_repeat attribute. This PR extends it to compute tiles as well.

Specifically:

  • the objectfifo attribute name is changed from memtile_repeat to repeat_count,
  • currently the objectfifo attribute is lowered to the repeat_count attribute of DMAStartOps for Mem tiles: this is added for compute tiles,
  • currently a repeat value can also be specified using data layout transformation dimensions in the objectfifo for Mem tiles: this is removed as the repeat dimension is not actually exposed in mlir-aie and this features (wrongly) assumed the highest dimension was the repeat dimension.

TODO: in the main branch, the memtile_repeat examples have been moved to test/npu-xrt/ : for me this leads to some failures locally that I don't see in the CI - currently investigating

Copy link
Contributor

github-actions bot commented Oct 18, 2024

Coverage Report

Created: 2024-10-18 16:39

Click here for information about interpreting this report.

FilenameFunction CoverageLine CoverageRegion CoverageBranch Coverage
IR/AIEDialect.cpp 89.63% 82.21% 84.72% 74.71%
Transforms/AIEObjectFifoStatefulTransform.cpp 100.00% 93.92% 91.76% 85.66%
Totals 92.05% 87.78% 87.24% 79.01%
Generated by llvm-cov -- llvm version 14.0.0

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.

2 participants