Skip to content

Remove CONTEXT_INFO_ATOMIC queries. #2374

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Remove CONTEXT_INFO_ATOMIC queries. #2374

wants to merge 1 commit into from

Conversation

@aarongreig
Copy link
Contributor

@aarongreig aarongreig commented Nov 22, 2024

These were used as dummy values in the sycl runtime to implement meta-queries for each device in the given context. The affiliated LLVM change for this PR updates that to use __SYCL_TRAIT_HANDLED_IN_RT instead.

Fixes #2119

@aarongreig
Copy link
Contributor Author

aarongreig commented Nov 22, 2024

LLVM PR intel/llvm#16160

@github-actions github-actions bot added loader Loader related feature/bug conformance Conformance test suite issues. specification Changes or additions to the specification level-zero L0 adapter specific issues cuda CUDA adapter specific issues hip HIP adapter specific issues opencl OpenCL adapter specific issues native-cpu Native CPU adapter specific issues labels Nov 22, 2024
@aarongreig aarongreig force-pushed the aaron/removeContextAtomicQueries branch from 0a8855e to 76d8873 Compare November 25, 2024 16:26
@aarongreig aarongreig marked this pull request as ready for review November 25, 2024 16:27
@aarongreig aarongreig requested review from a team as code owners November 25, 2024 16:27
@aarongreig aarongreig requested a review from jchlanda November 25, 2024 16:27
jchlanda
jchlanda approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@jchlanda jchlanda left a comment

Choose a reason for hiding this comment

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

Cuda/HIP lgtm 👍

nrspruit
nrspruit approved these changes Dec 6, 2024
View reviewed changes
Copy link
Contributor

@nrspruit nrspruit left a comment

Choose a reason for hiding this comment

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

L0 changes LGTM

@aarongreig aarongreig force-pushed the aaron/removeContextAtomicQueries branch from 76d8873 to 855a4d1 Compare January 22, 2025 12:14
@aarongreig aarongreig force-pushed the aaron/removeContextAtomicQueries branch from 855a4d1 to f3fdd84 Compare February 4, 2025 15:42
@aarongreig aarongreig added the ready to merge Added to PR's which are ready to merge label Feb 5, 2025
@martygrant
Copy link
Contributor

martygrant commented Feb 20, 2025

Unified Runtime -> intel/llvm Repo Move Notice

Information

The source code of Unified Runtime has been moved to intel/llvm under the unified-runtime top-level directory,
all future development will now be carried out there. This was done in intel/llvm#17043.

The code will be mirrored to oneapi-src/unified-runtime and the specification will continue to be hosted at oneapi-src.github.io/unified-runtime.

The contribution guide has been updated with new instructions for contributing to Unified Runtime.

PR Migration

All open PRs including this one will be labelled auto-close and shall be automatically closed after 30 days.
To allow for some breathing space, this automation will not be enabled until next week (27/02/2025).

Should you wish to continue with your PR you will need to migrate it to intel/llvm.
We have provided a script to help automate this process.

This is an automated comment.

@aarongreig
Remove CONTEXT_INFO_ATOMIC queries.
097a1ca 
These were used as dummy values in the sycl runtime to implement
meta-queries for each device in the given context. The affiliated LLVM
change for this PR updates that to use __SYCL_TRAIT_HANDLED_IN_RT
instead.

Fixes oneapi-src#2119
@aarongreig aarongreig force-pushed the aaron/removeContextAtomicQueries branch from f3fdd84 to 097a1ca Compare February 24, 2025 14:39
@aarongreig aarongreig closed this Feb 24, 2025
@aarongreig aarongreig mentioned this pull request Feb 24, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nrspruit nrspruit nrspruit approved these changes

+4 more reviewers

@jchlanda jchlanda jchlanda approved these changes

@hvdijk hvdijk hvdijk approved these changes

@kbenzie kbenzie kbenzie approved these changes

@martygrant martygrant martygrant approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

conformance Conformance test suite issues. cuda CUDA adapter specific issues hip HIP adapter specific issues level-zero L0 adapter specific issues loader Loader related feature/bug native-cpu Native CPU adapter specific issues opencl OpenCL adapter specific issues ready to merge Added to PR's which are ready to merge specification Changes or additions to the specification

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove CONTEXT_INFO_ATOMIC_ queries.

6 participants

@aarongreig @martygrant @jchlanda @hvdijk @kbenzie @nrspruit