Skip to content

[Unity][Dlight] Support dlight gemv rule on nested inner block #16251

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.

Sign up for GitHub

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

Jump to bottom
Merged
jinhongyii merged 1 commit into from
Jan 5, 2024
Merged

[Unity][Dlight] Support dlight gemv rule on nested inner block #16251

jinhongyii merged 1 commit into from
Jan 5, 2024

Conversation

@jinhongyii
Copy link
Contributor

@jinhongyii jinhongyii commented Dec 15, 2023 •
edited
Loading

  1. When user wants to partly specialize schedule, they can create an inner block and apply dlight on it. In the meanwhile, the user is free to schedule outer block. This PR supports such usecase on gemv rule.
  2. Fix loop order bugs in reduction rule.

@jinhongyii
Copy link
Contributor Author

jinhongyii commented Dec 15, 2023

cc: @tqchen @spectrometerHBH

@junrushao junrushao force-pushed the unity branch 2 times, most recently from c95d45f to 45eeb8c Compare December 18, 2023 21:00
@jinhongyii jinhongyii force-pushed the dlight-support-blockize branch from 0723897 to f5888da Compare December 29, 2023 22:17
@jinhongyii jinhongyii force-pushed the dlight-support-blockize branch from 75b0198 to bc9d883 Compare January 4, 2024 18:25
@jinhongyii jinhongyii force-pushed the dlight-support-blockize branch from bc9d883 to ad5052f Compare January 4, 2024 19:33
@jinhongyii jinhongyii merged commit 31659b6 into apache:unity Jan 5, 2024
@ysh329 ysh329 mentioned this pull request Apr 21, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@spectrometerHBH spectrometerHBH spectrometerHBH approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jinhongyii @spectrometerHBH