Skip to content

Add common pattern fusion#135

Merged
tancheng merged 10 commits intocoredac:mainfrom
HobbitQia:main
Oct 9, 2025
Merged

Add common pattern fusion#135
tancheng merged 10 commits intocoredac:mainfrom
HobbitQia:main

Conversation

@HobbitQia
Copy link
Copy Markdown
Collaborator

Fuse gep + load/store, mul + add in FusePatternPass which I observed them in most kernels and provide a C++ test case to verify the correctness.

For now, the fusion should be written manually, I am working on searching on common patterns based on frequency and fusing different pattern into the same op but with different attributes to differentiate each other. I will finish this these days.

@tancheng
Copy link
Copy Markdown
Contributor

fusing different pattern into the same op but with different attributes to differentiate each other.

This sounds a good idea, which is different from creating every neura.new_op for each fused op. However, we then need to carefully put sufficient/many attributes to "differentiate each other". Anyways, the design about those attributes can be first prototyped in a simple way and then improved gradually.

@tancheng
Copy link
Copy Markdown
Contributor

tancheng commented Oct 6, 2025

Looks like the merge main into main deletes all code changes? my bad, sth wrong with my github

@tancheng tancheng merged commit 2eaa9c7 into coredac:main Oct 9, 2025
1 check passed
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