Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][ESIMD] Add verifications that device has necessary aspects #839

Merged
merged 5 commits into from
Feb 24, 2022

Conversation

vasilytric
Copy link

Provide verifications that device has necessary aspects to interact with double or sycl::half data types, if device doesn't has necessary aspects such data type will be skipped during runtime

@yuriykoch
Copy link

Thanks, Vasily

- Rename should_run_test_with_current_datatype with should_skip_test_with
- Add comment to should_skip_test_with function
- Change should_skip_test_with function signature: obtain reference to device
  instead of rvalue device
@vasilytric
Copy link
Author

@kbobrovs Would you mind to take a look?

v-klochkov
v-klochkov previously approved these changes Feb 22, 2022
@vasilytric
Copy link
Author

@v-klochkov all comments has been applied, would you mind to merge this PR?

@v-klochkov v-klochkov merged commit dacacdf into intel:intel Feb 24, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
* [SYCL][ESIMD] Add verifications that device has necessary aspects
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
* [SYCL][ESIMD] Add verifications that device has necessary aspects
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…m-test-suite#839)

* [SYCL][ESIMD] Add verifications that device has necessary aspects
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants