Skip to content

Conversation

@Icohedron
Copy link
Contributor

@Icohedron Icohedron commented Nov 28, 2025

This PR removes the Clang && Vulkan XFAIL from Feature/CBuffer/array-dynamic-index.test since the test now XPASSes on all runners:

╭───┬──────────────────────┬─────────────┬─────────────────────────────┬────────┬──────────────────────────────────────────╮
│ # │      timestamp       │   run-id    │          workflow           │ status │                   test                   │
├───┼──────────────────────┼─────────────┼─────────────────────────────┼────────┼──────────────────────────────────────────┤
│ 0 │ 2025-11-28T12:08:27Z │ 19763375140 │ Windows Vulkan AMD Clang    │ XPASS  │ Feature/CBuffer/array-dynamic-index.test │
│ 1 │ 2025-11-28T18:05:22Z │ 19771042891 │ Windows Vulkan Intel Clang  │ XPASS  │ Feature/CBuffer/array-dynamic-index.test │
│ 2 │ 2025-11-28T04:03:18Z │ 19753735507 │ Windows Vulkan NVIDIA Clang │ XPASS  │ Feature/CBuffer/array-dynamic-index.test │
│ 3 │ 2025-11-28T18:09:04Z │ 19771107957 │ Windows Vulkan QC Clang     │ XPASS  │ Feature/CBuffer/array-dynamic-index.test │
╰───┴──────────────────────┴─────────────┴─────────────────────────────┴────────┴──────────────────────────────────────────╯

Before merging this PR, check/update the status of the original issue associated with the XFAIL: llvm/llvm-project#159602

@Icohedron Icohedron requested a review from s-perron November 28, 2025 19:45
@Icohedron Icohedron added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label Nov 28, 2025
Copy link
Contributor

@s-perron s-perron left a comment

Choose a reason for hiding this comment

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

That seems right. I just merged some fixed for cbuffers yesterday.

@Icohedron Icohedron merged commit 5c1e793 into llvm:main Dec 1, 2025
20 of 26 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.

3 participants