Skip to content

[SYCL] Remove deprecated sub_group::load and sub_group::store#21875

Merged
againull merged 3 commits intointel:syclfrom
againull:remove_deprecated_load_store
Apr 27, 2026
Merged

[SYCL] Remove deprecated sub_group::load and sub_group::store#21875
againull merged 3 commits intointel:syclfrom
againull:remove_deprecated_load_store

Conversation

@againull
Copy link
Copy Markdown
Contributor

These methods were deprecated in #15405 in favor of sycl::ext::oneapi::experimental::group_load/group_store.
Moved deprecated group_load_store spec to removed.

These methods were deprecated in intel#15405 in favor of
sycl::ext::oneapi::experimental::group_load/group_store.

Moved deprecated group_load_store spec to removed.
@againull againull requested review from a team as code owners April 25, 2026 00:31
@againull againull requested a review from slawekptak April 25, 2026 00:31
Comment thread sycl/test/check_device_code/matrix/matrix_load_store_as.cpp Outdated
Copy link
Copy Markdown
Contributor

@YuriPlyakhin YuriPlyakhin left a comment

Choose a reason for hiding this comment

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

spec file move LGTM

@againull againull requested a review from dkhaldi April 27, 2026 17:35
Copy link
Copy Markdown
Contributor

@dkhaldi dkhaldi left a comment

Choose a reason for hiding this comment

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

LGTM

@gmlueck
Copy link
Copy Markdown
Contributor

gmlueck commented Apr 27, 2026

I'm very happy to remove this, but I thought we were not able to remove APIs in a minor release. I vaguely remember there being a special case, though. Was it this API that we decided to remove because it had been deprecated so long and because we forgot to remove it in the last major release?

@againull
Copy link
Copy Markdown
Contributor Author

I'm very happy to remove this, but I thought we were not able to remove APIs in a minor release. I vaguely remember there being a special case, though. Was it this API that we decided to remove because it had been deprecated so long and because we forgot to remove it in the last major release?

Yes, that's correct, it was decided to remove this API as an exception.

@againull againull merged commit 4adef62 into intel:sycl Apr 27, 2026
31 checks passed
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