-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL] Set specialization constants in sycl::compile #8166
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
[SYCL] Set specialization constants in sycl::compile #8166
Conversation
This commit makes sycl::compile set specialization constants. The functionailty for this is combined with that used by sycl::build. Signed-off-by: Larsen, Steffen <[email protected]>
} | ||
} | ||
} | ||
setSpecializationConstants(InputImpl, NativePrg, Plugin); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we fold enableITTAnnotationsIfNeeded(NativePrg, Plugin)
into the helper as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We cannot fully merge it as it is used in another place that sets specialization constants differently. However, we may as well call it inside setSpecializationConstants
.
Signed-off-by: Larsen, Steffen <[email protected]>
/testwin |
/verify with intel/llvm-test-suite#1560 |
Signed-off-by: Larsen, Steffen <[email protected]>
/verify with intel/llvm-test-suite#1560 |
Signed-off-by: Larsen, Steffen <[email protected]>
This commit makes
sycl::compile
set specialization constants. The functionality for this is combined with that used bysycl::build
.