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

Conversation

v-klochkov
Copy link

The corresponding change in SYCL headers: intel/llvm/4718

Signed-off-by: Vyacheslav N Klochkov [email protected]

The corresponding change in SYCL headers: intel/llvm/4718

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
return 1;
advanceId(Id, NDR.get_global_range());
}
return 0;
Copy link

Choose a reason for hiding this comment

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

It would be good to cleanup - sycl::free the allocated memory

Copy link
Author

Choose a reason for hiding this comment

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

Thank you. Fixed in ba6e25f

@v-klochkov v-klochkov requested a review from kbobrovs October 12, 2021 05:10
@kbobrovs kbobrovs merged commit 13827f2 into intel:intel Oct 12, 2021
@v-klochkov v-klochkov deleted the esimd_spirv_globals branch October 12, 2021 05:14
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
[SYCL][Matrix] cherry pick from intel#645
Add XFAIL to the ops test because of bug
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…lvm-test-suite#503)

* [ESIMD] Add a test to check loads from SPIRV builtin globals

The corresponding change in SYCL headers: intel/llvm/4718

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
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