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

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

Merged
merged 3 commits into from
Oct 12, 2021

Conversation

v-klochkov
Copy link

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

Signed-off-by: Vyacheslav N Klochkov vyacheslav.n.klochkov@intel.com

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

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
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 <vyacheslav.n.klochkov@intel.com>
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