Skip to content

Move Lavapipe targets to experimental tier#1371

Merged
kmpeng merged 1 commit into
llvm:mainfrom
kmpeng:move-lavapipe-experimental
Jul 14, 2026
Merged

Move Lavapipe targets to experimental tier#1371
kmpeng merged 1 commit into
llvm:mainfrom
kmpeng:move-lavapipe-experimental

Conversation

@kmpeng

@kmpeng kmpeng commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Lavapipe targets were added as tier 1 targets in #1359, but since they're not completely passing yet, we should move them to the experimental tier.

@kmpeng kmpeng added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label Jul 14, 2026
matrix:
SKU: [windows-amd, windows-nvidia, windows-qc]
TestTarget: [check-hlsl-vk, check-hlsl-clang-vk]
TestTarget: [check-hlsl-vk, check-hlsl-clang-vk, check-hlsl-vk-lavapipe, check-hlsl-clang-vk-lavapipe]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added Lavapipe to test-vk-all in addition to test-all, not sure if that's right

@kmpeng kmpeng requested a review from joaosaffran July 14, 2026 18:43
exclude:
- { SKU: windows-nvidia, TestTarget: check-hlsl-d3d12 }
- { SKU: windows-nvidia, TestTarget: check-hlsl-clang-d3d12 }
- { SKU: windows-nvidia, TestTarget: check-hlsl-vk-lavapipe }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be clear, I think this means we're excluding NVIDIA lavapipe targets from the extra / experimental tier.
Which means they work well. Is that right?

@kmpeng kmpeng Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are "excluding" them from the experimental tier because there's no nvidia lavapipe config. Since the matrix is a SKU × TestTarget cross-product and windows-nvidia is needed for other targets, the excludes are just to drop the generated nvidia × lavapipe combos

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah that wasn't clear I thought we were excluding windows-nvidia because we only needed one x86 SKU covered by windows-amd and one aarch64 sku covered by windows-qc.

@kmpeng kmpeng merged commit 6dc8d4a into llvm:main Jul 14, 2026
43 of 54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-all When applied to a PR this will opt-in to additional pre-merge test configurations..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants