Skip to content

Conversation

@Icohedron
Copy link
Contributor

@Icohedron Icohedron commented Dec 4, 2025

The two tests are XPASSing on all machines except Intel under Vulkan with Clang. See:

But I believe the existing linked issue for the XFAIL is still the most relevant one so I did not change the linked bug.
#591 and #592 can probably be closed as duplicates of #599.

@Icohedron Icohedron requested a review from farzonl December 4, 2025 21:49
@Icohedron Icohedron added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label Dec 4, 2025
@farzonl
Copy link
Member

farzonl commented Dec 4, 2025

hmm if this is a driver bug maybe it shouldn't be in https://github.com/llvm/llvm-project and instead we should close #599.

@Icohedron
Copy link
Contributor Author

Icohedron commented Dec 4, 2025

hmm if this is a driver bug maybe it shouldn't be in https://github.com/llvm/llvm-project and instead we should close llvm/offload-test-suite#599.

We can also transfer the issue from llvm-project to the offload-test-suite repo. I believe the issue link will also redirect to the new one if we do that.

edit: I transferred the issue and I have made a commit to change the link to point to the new issue under the offload-test-suite. (which isn't necessary, as the old link redirects to the new one, but I still did it anyways).

@Icohedron Icohedron merged commit 2b96b58 into llvm:main Dec 5, 2025
9 of 23 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.

2 participants