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

[SYCL] Refactor invalid use of local accessor #1646

Merged
merged 13 commits into from
Mar 24, 2023

Conversation

mmoadeli
Copy link

@mmoadeli mmoadeli commented Mar 9, 2023

According to local accessors of the SYCL specification, a local accessor must not be used in a SYCL kernel function that is invoked via single_task or via the simple form of parallel_for that takes a range parameter.

  • Update invalid use of local accessors.
  • Add test to catch thrown exception

@mmoadeli mmoadeli requested a review from a team as a code owner March 9, 2023 10:39
@mmoadeli mmoadeli requested a review from steffenlarsen March 9, 2023 10:39
@mmoadeli
Copy link
Author

mmoadeli commented Mar 9, 2023

/Verify with intel/llvm#8581

Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

LGTM!

@mmoadeli mmoadeli changed the title Diagnostic on using local accessor in single_task or parallel_for(range) Refactor invalid use of local accessor Mar 23, 2023
@mmoadeli mmoadeli changed the title Refactor invalid use of local accessor [SYCL] Refactor invalid use of local accessor Mar 23, 2023
@steffenlarsen
Copy link

Verified through intel/llvm#8581.

@steffenlarsen steffenlarsen merged commit d578db9 into intel:intel Mar 24, 2023
bader added a commit that referenced this pull request Mar 24, 2023
bader added a commit that referenced this pull request Mar 24, 2023
Revert "[SYCL] Refactor invalid use of local accessor (#1646)"

This reverts commit d578db9.
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…1646)

According to [local accessors](https://registry.khronos.org/SYCL/specs/sycl-2020/html/sycl-2020.html#sec:accessor.local) of the SYCL specification, a local accessor must not be used in a SYCL kernel function that is invoked via single_task or via the simple form of parallel_for that takes a range parameter.
* Update invalid use of local accessors.
* Add test to catch thrown exception
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…st-suite#1695)

Revert "[SYCL] Refactor invalid use of local accessor (intel/llvm-test-suite#1646)"

This reverts commit c257c0b.
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