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

[SYCL][NFC] Modernize Basic/info.cpp #1558

Merged

Conversation

AlexeySachkov
Copy link

Replaced usages of cl_* types with data types listed in SYCL 2020 spec.

The only exception is info::reference_count: I can't find this query in the spec and therefore left cl_* type in place, but prefixed it with opencl:: namespace so it matches its defintion in SYCL 2020.

Replaced usages of `cl_*` types with data types listed in SYCL 2020
spec.

The only exception is `info::reference_count`: I can't find this query
in the spec and therefore left `cl_*` type in place, but prefixed it
with `opencl::` namespace so it matches its defintion in SYCL 2020.
@AlexeySachkov
Copy link
Author

@intel/llvm-reviewers-runtime, could you please take a look?

@steffenlarsen steffenlarsen merged commit f0da2bb into intel:intel Feb 2, 2023
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
Replaced usages of `cl_*` types with data types listed in SYCL 2020
spec.

The only exception is `info::reference_count`: I can't find this query
in the spec and therefore left `cl_*` type in place, but prefixed it
with `opencl::` namespace so it matches its defintion in SYCL 2020.
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Replaced usages of `cl_*` types with data types listed in SYCL 2020
spec.

The only exception is `info::reference_count`: I can't find this query
in the spec and therefore left `cl_*` type in place, but prefixed it
with `opencl::` namespace so it matches its defintion in SYCL 2020.
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.

2 participants