Skip to content

[SYCL] Remove sycl_ext_oneapi_default_context in preview-breaking mode - #22769

Open
softwaresat wants to merge 4 commits into
intel:syclfrom
softwaresat:remove-oneapi-default-context
Open

[SYCL] Remove sycl_ext_oneapi_default_context in preview-breaking mode#22769
softwaresat wants to merge 4 commits into
intel:syclfrom
softwaresat:remove-oneapi-default-context

Conversation

@softwaresat

@softwaresat softwaresat commented Jul 26, 2026

Copy link
Copy Markdown

Removes sycl_ext_oneapi_default_context in preview-breaking mode.

  • Guards platform::ext_oneapi_get_default_context() and
    SYCL_EXT_ONEAPI_DEFAULT_CONTEXT with
    __INTEL_PREVIEW_BREAKING_CHANGES.
  • Preserves platform::khr_get_default_context() and the supported
    device::ext_oneapi_get_default_context() API.
  • Updates the affected unit tests.
  • Adds a reminder to move the extension specification to removed/
    when the API is removed from normal builds.

Addresses the sycl_ext_oneapi_default_context portion of #22581.

@softwaresat
softwaresat requested review from a team as code owners July 26, 2026 05:16

@Robertkq Robertkq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, good work!
Only note: In this repository, the body of the PR usually becomes the commit message of the Squash & Merge procedure when the PR is eventually merged, as such I think you should try to keep the body of the PR as concise as possible on the changes made and to add comments to the PR for explaining the decisions behind them

Comment thread sycl/include/sycl/platform.hpp Outdated
Comment thread sycl/doc/extensions/removed/sycl_ext_oneapi_default_context.asciidoc Outdated
Comment thread sycl/test/extensions/removed_default_context.cpp Outdated
Comment thread sycl/source/platform.cpp Outdated
Comment thread sycl/unittests/queue/DeviceCheck.cpp Outdated
@softwaresat
softwaresat force-pushed the remove-oneapi-default-context branch from 177ca06 to f11e796 Compare July 29, 2026 23:18
@KornevNikita

Copy link
Copy Markdown
Contributor

@KseniyaTikhomirova could you please take a look?

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.

5 participants