Skip to content

[SYCL] Disable launch_policy_neg.cpp on Windows #17367

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 9, 2025
Merged

Conversation

sarnex
Copy link
Contributor

@sarnex sarnex commented Mar 8, 2025

Fails to compile on new MSVC.

@sarnex sarnex temporarily deployed to WindowsCILock March 8, 2025 21:46 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 8, 2025 22:15 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 8, 2025 22:15 — with GitHub Actions Inactive
@sarnex sarnex marked this pull request as ready for review March 9, 2025 00:33
@sarnex sarnex requested a review from a team as a code owner March 9, 2025 00:33
@sarnex
Copy link
Contributor Author

sarnex commented Mar 9, 2025

Merging now because I need this to update the compiler which I can only do when nobody is using CI

@sarnex sarnex merged commit 1a9a282 into intel:sycl Mar 9, 2025
21 of 23 checks passed
adamfidel pushed a commit to reble/llvm that referenced this pull request Mar 11, 2025
Fails to compile on new MSVC.

Signed-off-by: Sarnie, Nick <[email protected]>
AlexeySachkov pushed a commit to AlexeySachkov/llvm that referenced this pull request Apr 2, 2025
Fails to compile on new MSVC.

Signed-off-by: Sarnie, Nick <[email protected]>
AlexeySachkov added a commit that referenced this pull request Apr 3, 2025
This is a joined cherry-pick of #17367, #17362 and
#17746 intended to fix a few remaining issues we see in 6.1
nightly validation run.

Descriptions of cherry-picked commits:

---

[SYCL] Disable launch_policy_neg.cpp on Windows (#17367)

Fails to compile on new MSVC.

Patch-by: Sarnie, Nick <[email protected]>

---

[SYCL] Avoid testing race condition

Use no output file instead of default output file in LIT tests.
This occasionally caused errors on windows when multiple files compile
to the same output file. Using `-fsyntax-only` hjere also avoids running
the full compiler pipeline

See #7872 for more context

Patch-By: David Garcia Orozco <[email protected]>

---

[SYCL] Fix clang-linker-wrapper Windows test (#17746)

Failing now because I recently added `ptxas` to the path on these
runners.

Patch-by: Sarnie, Nick <[email protected]>

---
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.

1 participant