Skip to content

SYCL] Support specialization constants in the L0 plugin #1982

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 2, 2020

Conversation

againull
Copy link
Contributor

Author: Jaime Arteaga [email protected]
Signed-off-by: Artur Gainullin [email protected]

smaslov-intel and others added 2 commits June 24, 2020 14:07
- Defer L0 module creation/build until piProgramBuild/piProgramCompile.
- Support build log extraction.
- Support build options.

Author: Sergey V Maslov <[email protected]>
Signed-off-by: Artur Gainullin <[email protected]>
@againull againull requested a review from smaslov-intel as a code owner June 24, 2020 21:57
@againull
Copy link
Contributor Author

Please do not review the first commit, it is covered by another PR.

Also, we need to wait when GPU driver version will be >= https://github.com/intel/compute-runtime/releases/tag/20.22.16952 to enable spec constant LIT tests They are currently disabled for opencl also.

@againull
Copy link
Contributor Author

@smaslov-intel Could you please review.

smaslov-intel
smaslov-intel previously approved these changes Jul 1, 2020
@againull againull merged commit be4e641 into intel:sycl Jul 2, 2020
@againull againull deleted the l0_specialization_consts branch December 3, 2022 00:02
againull pushed a commit to againull/llvm that referenced this pull request May 4, 2023
Before reading Target function name operand of `DebugFunction` we need
to make sure it's the translation of appropriate debug extension.
Otherwise, we can get in a situation, where we do have 11 operands, but
the last one is not a `String` with name, but the `DebugFunctionDeclaration`.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@f990940
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.

3 participants