Skip to content

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