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

Remove info::platform::profile from the header #608

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

VerenaBeckham
Copy link
Contributor

It is never described in the spec.
Alternatively, we could bring the description in the spec back and mark it deprecated.

@gmlueck
Copy link
Contributor

gmlueck commented Aug 26, 2024

FWIW, DPC++ does seem to implement this, but I'm still in favor of removing it from the spec.

I think this is a holdover from OpenCL, which supports the CL_PLATFORM_PROFILE query.

@TApplencourt
Copy link
Contributor

CI failure is due to new the "PR to CTS" thingy step

@VerenaBeckham
Copy link
Contributor Author

@psalz could you please have a look at the CI failure?

@psalz
Copy link
Member

psalz commented Aug 27, 2024

@psalz could you please have a look at the CI failure?

Hmm, might've been spurious. Re-running the job fixed it 🤷‍♂️

Edit: Nevermind, I believe it's because you're not a member of the @KhronosGroup organization, the job failed to assign the issue to you. I will look into that!

@tomdeakin tomdeakin added the Agenda To be discussed during a SYCL committee meeting label Aug 28, 2024
@tomdeakin
Copy link
Contributor

Waiting on CTS fix.

@VerenaBeckham
Copy link
Contributor Author

The CTS fix has been implemented and merged. This can now also be merged.

@tomdeakin
Copy link
Contributor

Wg approved to merge.

@tomdeakin tomdeakin merged commit 88f0d48 into KhronosGroup:main Sep 26, 2024
3 checks passed
gmlueck pushed a commit to gmlueck/SYCL-Docs that referenced this pull request Sep 26, 2024
Cherry pick KhronosGroup#608 from main
(cherry picked from commit 88f0d48)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda To be discussed during a SYCL committee meeting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants