NFC: replace deprecated API: cute.make_fragment - #3473
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR migrates epilogue register tensor constructions from ChangesRegister Tensor API Migration in SM100 Kernels
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review
This pull request replaces several calls to cute.make_fragment with cute.make_rmem_tensor in the GEMM kernels dense_blockscaled_gemm_sm100.py and grouped_gemm_masked_blackwell.py. No review comments were provided, and there is no additional feedback.
|
/bot run /tests/gemm |
|
[FAILED] Pipeline #53265395: 9/20 passed |
📌 Description
More mechanical change to replace deprecated API:
🔍 Related Issues
N/A
🚀 Pull Request Checklist
Thank you for contributing to FlashInfer! Before we review your pull request, please make sure the following items are complete.
✅ Pre-commit Checks
pre-commitby runningpip install pre-commit(or used your preferred method).pre-commit install.pre-commit run --all-filesand fixed any reported issues.N/A
🧪 Tests
unittest, etc.).N/A
Reviewer Notes
N/A
Summary by CodeRabbit