-
Notifications
You must be signed in to change notification settings - Fork 790
[SYCL] Skip device diagnostics for unevaluated functions. #1093
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
Conversation
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.
I'm reasonably OK with this... I fear that we will have some errors that the evaluated context doesn't matter. I can't think of any active ones at the moment, but perhaps the other reviewers can help.
Otherwise, some language/formatting issues.
0e7c4ab
to
60c68e5
Compare
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.
LGTM, but I'd like Erich to approve before merging.
Do not produce an error if some restricted feature (e.g. exceptions) is used by a function only referenced in unevaluated context(e.g. in decltype expression) In order to do that, skip such functions when extracting device part of the code. Signed-off-by: Ilya Stepykin <[email protected]>
60c68e5
to
7e0881e
Compare
Fixed kernel bundle related tests. The following tests were just missing SYCL triple in the compile commands: KernelAndProgram/kernel-bundle-merge-options.cpp KernelAndProgram/kernel-bundle-merge-options-env.cpp KernelAndProgram/multiple-kernel-linking.cpp The following tests are checking that compilation errors are reported during runtime, but for CUDA and HIP these are actually reported at compile time. So these were marked as unsupported and a comment was added mentioning that: KernelAndProgram/cache-build-result.cpp KernelAndProgram/build-log.cpp KernelAndProgram/undefined-symbol.cpp
Do not produce an error if some restricted feature
(e.g. exceptions) is used by a function only referenced
in unevaluated context(e.g. in decltype expression)
In order to do that, skip such functions when extracting
device part of the code.
Signed-off-by: Ilya Stepykin [email protected]