Skip to content

[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

Merged
merged 1 commit into from
Feb 4, 2020

Conversation

ilyastepykin
Copy link
Contributor

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]

Copy link
Contributor

@erichkeane erichkeane left a 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.

Copy link
Contributor

@bader bader left a 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]>
@bader bader merged commit 5eae571 into intel:sycl Feb 4, 2020
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Feb 23, 2023
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
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.

4 participants