Repository navigation
cuda-oxide: update to nightly-2026-10-11 - #1482
Merged
Merged
Conversation
Signed-off-by: nihalp <nihalp@nvidia.com>
Signed-off-by: nihalp <nihalp@nvidia.com>
Signed-off-by: nihalp <nihalp@nvidia.com>
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
2 tasks done
Signed-off-by: nihalp <nihalp@nvidia.com>
Signed-off-by: nihalp <nihalp@nvidia.com>
Signed-off-by: nihalp <nihalp@nvidia.com>
nihalpasham
marked this pull request as ready for review
October 11, 2026 10:44
nihalpasham
added a commit
that referenced
this pull request
Oct 11, 2026
Fix `sparse_mma_fp8`, which tested twelve metadata encodings even though plain FP8 `mma.sp` permits only six. The invalid encodings caused 5,104 mismatches on RTX 5090. The example and layout probe now use the six valid encodings. Inline comments and generated safety docs explain the restriction and link to the [PTX specification](https://docs.nvidia.com/cuda/archive/12.4.1/parallel-thread-execution/index.html#warp-level-sparse-matrix-storage). Instruction lowering and ABI are unchanged. Also set `rust-analyzer.cargo.features` to `[]` so VS Code stops enabling every optional feature. ## Validation - Both regressions fail before the fix; 319 generator tests and 3 example tests pass. - Both RTX 5090s pass all 6,144 results, with zero memory or synchronization errors. - Verified with both the old nightly and `nightly-2026-10-11` after #1482; generated outputs and all 1,029 ABI entries pass. - Before the editor-only update: CI had 54 passes and 6 expected skips; formatting and strict clippy passed. The updated settings file validates; new CI results are pending. Follow-up to #1295. Running examples on GPUs in CI remains separate work. --------- Signed-off-by: nihalp <nihalp@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update cuda-oxide from
nightly-2026-08-28tonightly-2026-10-11. This brings in rust-lang/rust#163242, needed for the separate kernel parameter-attribute work in #1483 (related to #1088 and #1098).Changes
core::intrinsics::reflection::type_idpath in the kernel-name helper.Validation
The existing scaffold issue—mixing registry and Git runtime types—also remains a separate follow-up.
Checklist