Skip to content
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

[SYCL][NFC] Re-enable sycl.c test on Windows #15433

Merged
merged 3 commits into from
Sep 20, 2024

Conversation

mdtoguchi
Copy link
Contributor

@mdtoguchi mdtoguchi commented Sep 18, 2024

The issue dealt with the resetting of the PATH environment variable. This was masking potential runtime dependencies (DLLs and such) for other tools being used when running the test. The solution is to append the PATH for the dummy aoc instead of overwriting.

Re-enabling for testing purposes.  Am unable to reproduce locally.
@mdtoguchi mdtoguchi marked this pull request as ready for review September 20, 2024 17:31
@mdtoguchi mdtoguchi requested a review from a team as a code owner September 20, 2024 17:31
@mdtoguchi
Copy link
Contributor Author

@intel/llvm-gatekeepers , this looks ready for merge, please take a look thanks!

@sarnex sarnex merged commit c0b49db into intel:sycl Sep 20, 2024
12 checks passed
@mdtoguchi mdtoguchi deleted the sycl-test-fail-win branch September 20, 2024 18:21
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.

3 participants