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

[TIR] Tuple Reduction Support in CreatePrimFunc #10671

Conversation

MasterJH5574
Copy link
Contributor

This PR adds the support of tuple reduction in CreatePrimFunc(..).

Prior to this PR, CreatePrimFunc(..) errored out when the input ReduceNode reduces on tuples.

With this PR, CreatePrimFunc(..) generates TIR blocks with correctness guarantees (under the help of LetStmt) when encountering such ReduceNodes. Please refer to the new unit tests for detailed generated TIR functions :-)


cc @hypercubestart @junrushao1994 @Hzfengsy @tqchen

Co-authored-by: Andrew Liu [email protected]

@junrushao
Copy link
Member

CC: @Hzfengsy Please review :-)

@MasterJH5574 MasterJH5574 force-pushed the feature/2022-03-17-create-primfunc-tuple-reduction branch from b6453da to 3eb989f Compare March 22, 2022 07:44
@github-actions github-actions bot requested a review from Hzfengsy March 22, 2022 07:45
@Hzfengsy Hzfengsy merged commit ae61603 into apache:main Mar 22, 2022
@junrushao
Copy link
Member

Thanks @hypercubestart @MasterJH5574 for getting the support for tuple reduction in CreatePrimFunc!!

pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
* [CreatePrimFunc] Support multi-source ReduceNode (apache#64)

* initial

* assert structural equal test

* Enhancement and tests

* Fix dtype

* Docs

Co-authored-by: Andrew Liu <[email protected]>
wrongtest-intellif pushed a commit that referenced this pull request Aug 30, 2022
* [CreatePrimFunc] Support multi-source ReduceNode (#64)

* initial

* assert structural equal test

* Enhancement and tests

* Fix dtype

* Docs

Co-authored-by: Andrew Liu <[email protected]>
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.

4 participants