Fix ci breakage for missing protection to is_exclusive - #605
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (23)
📝 WalkthroughWalkthroughThe change adds shared TCGEN05 wrappers for allocation, deallocation, and block-scaled MMA. Frost GEMM templates now use these wrappers instead of direct NVVM operations. Tests add architecture gating and AST validation for version-specific keyword forwarding. ChangesTCGEN05 helper migration
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This change adds guarded keyword handling across matmul templates to preserve CI and compatibility behavior, with limited expected production impact. Merge readiness is minimal; only a small follow-up remains to confirm the two new tests carry the required level marker. Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Before submitting
pre-commit runand committed any formatting changes.cat-*, one or moremod-*, and oneorig-*(see label list).Affected area
Summary
Add protection flag to is_exclusive to avoid ci breakage on Blackwell
Why
Related issues
API and compatibility impact
Testing
Summary by CodeRabbit
New Features
Bug Fixes
Tests