Skip to content

feat: add new optimized tutorial kernels - #4272

Merged
illsilin merged 4 commits into
developfrom
import/develop/ROCm_composable_kernel/pr-3676
Feb 17, 2026
Merged

feat: add new optimized tutorial kernels#4272
illsilin merged 4 commits into
developfrom
import/develop/ROCm_composable_kernel/pr-3676

Conversation

@assistant-librarian

@assistant-librarian assistant-librarian Bot commented Feb 3, 2026

Copy link
Copy Markdown
Contributor
  • Add 01_naive_gemm baseline implementation
  • Add 02_padding_k_first with PADDING_K_FIRST + MFMA_32x32x16
  • Add 03_mfma_16x16x16 with PADDING_K_FIRST + MFMA_16x16x16
  • Share common reference_gemm.hpp in parent gemm/ directory

Proposed changes

Please describe the motivation behind the pull request, whether it enables a new feature or fixes a bug. If there are associated pull requests or issues, please link them to the pull request.

Checklist

Please put an x into the boxes that apply. You can also fill these out after creating the PR. If you're not sure, please don't hesitate to ask.

  • I have added tests relevant to the introduced functionality, and the unit tests are passing locally
  • I have added the test to REGRESSION_TESTS list defined at the top of CMakeLists.txt in tests/CMakeLists.txt, IF the test takes more than 30 seconds to run.
  • I have added inline documentation which enables the maintainers with understanding the motivation
  • I have removed the stale documentation which is no longer relevant after this pull request
  • (If this change is user-facing) I have added release notes which provide the end users with a brief summary of the improvement from this pull request
  • I have run clang-format on all changed files
  • Any dependent changes have been merged

Discussion

If this is a relatively large or complex change, feel free to start a discussion by explaining why you chose the solution you did and what alternatives you considered

🔁 Imported from ROCm/composable_kernel#3676
🧑‍💻 Originally authored by @AviralGoelAMD

AviralGoelAMD and others added 3 commits January 29, 2026 12:45
@AviralGoelAMD
feat: add new optimized tutorial kernels
adb8f67 
- Add 01_naive_gemm baseline implementation
- Add 02_padding_k_first with PADDING_K_FIRST + MFMA_32x32x16
- Add 03_mfma_16x16x16 with PADDING_K_FIRST + MFMA_16x16x16
- Share common reference_gemm.hpp in parent gemm/ directory
@AviralGoelAMD
Merge branch 'develop' into aviralgoel/gemm_tutorial
f96d74b
@systems-assistant
Merge commit 'f96d74b55da9274f7c448916a383f5567fff9016' into import/d…
296bb7e 
…evelop/ROCm_composable_kernel/pr-3676
@assistant-librarian assistant-librarian Bot added the external contribution Code contribution from users community.. label Feb 3, 2026
@illsilin
illsilin requested a review from a team as a code owner February 12, 2026 04:54
@illsilin
illsilin merged commit 52def72 into develop Feb 17, 2026
18 checks passed
@illsilin
illsilin deleted the import/develop/ROCm_composable_kernel/pr-3676 branch February 17, 2026 20:41
assistant-librarian Bot pushed a commit to ROCm/composable_kernel that referenced this pull request Feb 17, 2026
@AviralGoelAMD @assistant-librarian
[rocm-libraries] ROCm/rocm-libraries#4272 (commit 52def72)
1bf6600 
feat: add new optimized tutorial kernels
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add 01_naive_gemm baseline implementation
- Add 02_padding_k_first with PADDING_K_FIRST + MFMA_32x32x16
- Add 03_mfma_16x16x16 with PADDING_K_FIRST + MFMA_16x16x16
- Share common reference_gemm.hpp in parent gemm/ directory

## Proposed changes

Please describe the motivation behind the pull request, whether it
enables a new feature or fixes a bug. If there are associated pull
requests or issues, please link them to the pull request.

## Checklist

Please put an `x` into the boxes that apply. You can also fill these out
after creating the PR. If you're not sure, please don't hesitate to ask.

- [ ] I have added tests relevant to the introduced functionality, and
the unit tests are passing locally
- [ ] I have added the test to REGRESSION_TESTS list defined at the top
of CMakeLists.txt in tests/CMakeLists.txt, **IF** the test takes more
than 30 seconds to run.
- [ ] I have added inline documentation which enables the maintainers
with understanding the motivation
- [ ] I have removed the stale documentation which is no longer relevant
after this pull request
- [ ] (If this change is user-facing) I have added release notes which
provide the end users with a brief summary of the improvement from this
pull request
- [ ] I have run `clang-format` on all changed files
- [ ] Any dependent changes have been merged

## Discussion

If this is a relatively large or complex change, feel free to start a
discussion by explaining why you chose the solution you did and what
alternatives you considered
kamuruga08 pushed a commit that referenced this pull request Feb 19, 2026
@assistant-librarian @AviralGoelAMD
@systems-assistant @illsilin @kamuruga08
feat: add new optimized tutorial kernels (#4272)
be63d97 
- Add 01_naive_gemm baseline implementation
- Add 02_padding_k_first with PADDING_K_FIRST + MFMA_32x32x16
- Add 03_mfma_16x16x16 with PADDING_K_FIRST + MFMA_16x16x16
- Share common reference_gemm.hpp in parent gemm/ directory

## Proposed changes

Please describe the motivation behind the pull request, whether it
enables a new feature or fixes a bug. If there are associated pull
requests or issues, please link them to the pull request.

## Checklist

Please put an `x` into the boxes that apply. You can also fill these out
after creating the PR. If you're not sure, please don't hesitate to ask.

- [ ] I have added tests relevant to the introduced functionality, and
the unit tests are passing locally
- [ ] I have added the test to REGRESSION_TESTS list defined at the top
of CMakeLists.txt in tests/CMakeLists.txt, **IF** the test takes more
than 30 seconds to run.
- [ ] I have added inline documentation which enables the maintainers
with understanding the motivation
- [ ] I have removed the stale documentation which is no longer relevant
after this pull request
- [ ] (If this change is user-facing) I have added release notes which
provide the end users with a brief summary of the improvement from this
pull request
- [ ] I have run `clang-format` on all changed files
- [ ] Any dependent changes have been merged

## Discussion

If this is a relatively large or complex change, feel free to start a
discussion by explaining why you chose the solution you did and what
alternatives you considered



---
🔁 Imported from
[ROCm/composable_kernel#3676](ROCm/composable_kernel#3676)
🧑‍💻 Originally authored by @AviralGoelAMD

Co-authored-by: AviralGoelAMD <aviral.goel@amd.com>
Co-authored-by: systems-assistant[bot] <systems-assistant[bot]@users.noreply.github.com>
Co-authored-by: Illia Silin <98187287+illsilin@users.noreply.github.com>
kamuruga08 pushed a commit that referenced this pull request Feb 19, 2026
@assistant-librarian @AviralGoelAMD
@systems-assistant @illsilin @kamuruga08
feat: add new optimized tutorial kernels (#4272)
c16cf64 
- Add 01_naive_gemm baseline implementation
- Add 02_padding_k_first with PADDING_K_FIRST + MFMA_32x32x16
- Add 03_mfma_16x16x16 with PADDING_K_FIRST + MFMA_16x16x16
- Share common reference_gemm.hpp in parent gemm/ directory

## Proposed changes

Please describe the motivation behind the pull request, whether it
enables a new feature or fixes a bug. If there are associated pull
requests or issues, please link them to the pull request.

## Checklist

Please put an `x` into the boxes that apply. You can also fill these out
after creating the PR. If you're not sure, please don't hesitate to ask.

- [ ] I have added tests relevant to the introduced functionality, and
the unit tests are passing locally
- [ ] I have added the test to REGRESSION_TESTS list defined at the top
of CMakeLists.txt in tests/CMakeLists.txt, **IF** the test takes more
than 30 seconds to run.
- [ ] I have added inline documentation which enables the maintainers
with understanding the motivation
- [ ] I have removed the stale documentation which is no longer relevant
after this pull request
- [ ] (If this change is user-facing) I have added release notes which
provide the end users with a brief summary of the improvement from this
pull request
- [ ] I have run `clang-format` on all changed files
- [ ] Any dependent changes have been merged

## Discussion

If this is a relatively large or complex change, feel free to start a
discussion by explaining why you chose the solution you did and what
alternatives you considered



---
🔁 Imported from
[ROCm/composable_kernel#3676](ROCm/composable_kernel#3676)
🧑‍💻 Originally authored by @AviralGoelAMD

Co-authored-by: AviralGoelAMD <aviral.goel@amd.com>
Co-authored-by: systems-assistant[bot] <systems-assistant[bot]@users.noreply.github.com>
Co-authored-by: Illia Silin <98187287+illsilin@users.noreply.github.com>
jovanau pushed a commit to jovanau/rocm-libraries that referenced this pull request Mar 19, 2026
@assistant-librarian @AviralGoelAMD
@systems-assistant @illsilin @jovanau
feat: add new optimized tutorial kernels (ROCm#4272)
3596af1 
- Add 01_naive_gemm baseline implementation
- Add 02_padding_k_first with PADDING_K_FIRST + MFMA_32x32x16
- Add 03_mfma_16x16x16 with PADDING_K_FIRST + MFMA_16x16x16
- Share common reference_gemm.hpp in parent gemm/ directory

## Proposed changes

Please describe the motivation behind the pull request, whether it
enables a new feature or fixes a bug. If there are associated pull
requests or issues, please link them to the pull request.

## Checklist

Please put an `x` into the boxes that apply. You can also fill these out
after creating the PR. If you're not sure, please don't hesitate to ask.

- [ ] I have added tests relevant to the introduced functionality, and
the unit tests are passing locally
- [ ] I have added the test to REGRESSION_TESTS list defined at the top
of CMakeLists.txt in tests/CMakeLists.txt, **IF** the test takes more
than 30 seconds to run.
- [ ] I have added inline documentation which enables the maintainers
with understanding the motivation
- [ ] I have removed the stale documentation which is no longer relevant
after this pull request
- [ ] (If this change is user-facing) I have added release notes which
provide the end users with a brief summary of the improvement from this
pull request
- [ ] I have run `clang-format` on all changed files
- [ ] Any dependent changes have been merged

## Discussion

If this is a relatively large or complex change, feel free to start a
discussion by explaining why you chose the solution you did and what
alternatives you considered



---
🔁 Imported from
[ROCm/composable_kernel#3676](ROCm/composable_kernel#3676)
🧑‍💻 Originally authored by @AviralGoelAMD

Co-authored-by: AviralGoelAMD <aviral.goel@amd.com>
Co-authored-by: systems-assistant[bot] <systems-assistant[bot]@users.noreply.github.com>
Co-authored-by: Illia Silin <98187287+illsilin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@illsilin illsilin illsilin approved these changes
@carlushuang carlushuang Awaiting requested review from carlushuang
@qianfengz qianfengz Awaiting requested review from qianfengz
@aosewski aosewski Awaiting requested review from aosewski
@poyenc poyenc Awaiting requested review from poyenc
@geyyer geyyer Awaiting requested review from geyyer
@bartekxk bartekxk Awaiting requested review from bartekxk
@andriy-ca andriy-ca Awaiting requested review from andriy-ca
@afagaj afagaj Awaiting requested review from afagaj
@asleepzzz asleepzzz Awaiting requested review from asleepzzz
@tenpercent tenpercent Awaiting requested review from tenpercent
@ThomasNing ThomasNing Awaiting requested review from ThomasNing
@coderfeli coderfeli Awaiting requested review from coderfeli

Assignees

No one assigned

Labels

documentation external contribution Code contribution from users community.. imported pr project: composablekernel

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@illsilin @AviralGoelAMD