Skip to content

Preprations for controlflow fusion#79

Merged
ShangkunLi merged 2 commits intocoredac:mainfrom
ShangkunLi:pre-ctrl-fuse
Jul 19, 2025
Merged

Preprations for controlflow fusion#79
ShangkunLi merged 2 commits intocoredac:mainfrom
ShangkunLi:pre-ctrl-fuse

Conversation

@ShangkunLi
Copy link
Copy Markdown
Collaborator

In this pr:

  • Prepare different control flow kernels for the control flow fusion test in test/controflow-fuse/ (e.g., (non)-perfect nested loops, complex nested loop, nested-reduction loop)
  • Fix some bugs when trying to lower a non-perfect nested loop to dataflow
  • Provide both source .cpp and .mlirfiles of the tests

Copy link
Copy Markdown
Contributor

@tancheng tancheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ShangkunLi, it looks like we need to have our interpreter be done ASAP to verify the upcoming transformations.

@tancheng tancheng requested review from Yfeng-44 and itemkelvin July 15, 2025 15:41
@tancheng tancheng added the new feature New feature or request label Jul 15, 2025
@ShangkunLi
Copy link
Copy Markdown
Collaborator Author

Thanks @ShangkunLi, it looks like we need to have our interpreter be done ASAP to verify the upcoming transformations.

Agree. For now, I just manually go through each line of the generated dataflow ir to ensure the correctness.

@ShangkunLi ShangkunLi marked this pull request as ready for review July 15, 2025 16:31
@ShangkunLi ShangkunLi merged commit a0cd122 into coredac:main Jul 19, 2025
1 check passed
ShangkunLi added a commit that referenced this pull request Mar 12, 2026
Preprations for controlflow fusion
ShangkunLi added a commit that referenced this pull request Mar 12, 2026
Preprations for controlflow fusion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants