Skip to content

[TIRx][Test] Gate tcgen05.mma collector certification on nvcc >= 13.4 - #20270

Merged
spectrometerHBH merged 1 commit into
apache:mainfrom
spectrometerHBH:test/tirx-gate-tcgen05-collector-cert-on-nvcc-13-4
Sep 4, 2026
Merged

[TIRx][Test] Gate tcgen05.mma collector certification on nvcc >= 13.4#20270
spectrometerHBH merged 1 commit into
apache:mainfrom
spectrometerHBH:test/tirx-gate-tcgen05-collector-cert-on-nvcc-13-4

Conversation

@spectrometerHBH

Copy link
Copy Markdown
Contributor

Summary

test_ptx_tcgen05_mma_block_size_form assembles the collector-qualified tcgen05.mma ... block_scale forms at sm_107f. Those are PTX ISA 9.4 forms (#20266) and need the CUDA 13.4 ptxas; on a CUDA 13.2 host the test fails with Unknown modifier '.collector::b::fill' instead of skipping.

This moves that half into test_ptx_tcgen05_mma_block_size_collector_form, guarded by env.has_nvcc_version(13, 4), the same gate the whole-table certification tests already use. The sm_100a block32 certification and the mxf4 block16 legality check stay in the original test and keep running on older toolchains.

Testing

On a CUDA 13.2 host (B200):

tests/python/tirx/codegen/test_ptx_dialect.py -k tcgen05_mma_block_size
3 passed, 1 skipped (collector form: need nvcc >= 13.4)

The full tests/python/tirx/ suite on the same host previously had this as its only failure (2749 passed, 321 skipped).

The collector-qualified block_scale MMA forms are PTX ISA 9.4 and certify at
sm_107f, which needs the CUDA 13.4 ptxas. Move that half of
test_ptx_tcgen05_mma_block_size_form into its own test guarded by
env.has_nvcc_version(13, 4), the same gate the whole-table certification
tests already use, so older toolchains skip it instead of failing while the
sm_100a block32 certification and the block16 legality check still run.
@spectrometerHBH
spectrometerHBH merged commit 8ddd97d into apache:main Sep 4, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants