Skip to content

[SYCL] Move non device code checks outside of check_device_code #14175

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
Jun 27, 2024

Conversation

ianayl
Copy link
Contributor

@ianayl ianayl commented Jun 14, 2024

In the same vein as #14174, this PR moves test cases that do not check device code outside of the check_device_code folder. This is a separate PR as #14174 because the code is technically owned by another team.

@ianayl ianayl requested review from a team as code owners June 14, 2024 00:21
@ianayl ianayl requested a review from ldrumm June 14, 2024 00:21
@ianayl ianayl temporarily deployed to WindowsCILock June 17, 2024 17:44 — with GitHub Actions Inactive
@ianayl ianayl temporarily deployed to WindowsCILock June 17, 2024 19:18 — with GitHub Actions Inactive
@ianayl
Copy link
Contributor Author

ianayl commented Jun 19, 2024

@ldrumm Friendly ping, thank you!

Copy link
Contributor

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

Looks reasonable!

@ianayl
Copy link
Contributor Author

ianayl commented Jun 20, 2024

@intel/sycl-matrix-reviewers Friendly ping: just a quick change anyway.

Thanks in advance!

Copy link
Contributor

@YuriPlyakhin YuriPlyakhin left a comment

Choose a reason for hiding this comment

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

LGTM

@ianayl
Copy link
Contributor Author

ianayl commented Jun 21, 2024

@intel/llvm-gatekeepers This PR is ready, thank you!

@ianayl
Copy link
Contributor Author

ianayl commented Jun 27, 2024

@intel/llvm-gatekeepers Friendly ping, thanks!

@aelovikov-intel aelovikov-intel merged commit 666cf66 into intel:sycl Jun 27, 2024
14 checks passed
@ianayl ianayl deleted the move-non-device-code-checks branch July 3, 2024 17:37
aelovikov-intel pushed a commit that referenced this pull request Jul 8, 2024
)

It would appear that I missed a file in #14175. I looked over the files
again and did not catch anything else, but do let me know if there is
more.
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.

4 participants