Skip to content

[SYCL][LIBCLC][CUDA] Remove suq.depth instruction #5378

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
Jan 26, 2022

Conversation

pgorlani
Copy link
Contributor

This patch removes llvm.nvvm.suq.depth from
the image functions. This instruction doesn't
work in CUDA and causes a CUDA_ERROR_NOT_FOUND
or a CUDA_ERROR_NOT_SUPPORTED if present in the
fatbin, even if it is not actually executed.

This patch removes llvm.nvvm.suq.depth from
the image functions. This instruction doesn't
work in CUDA and causes a CUDA_ERROR_NOT_FOUND
or a CUDA_ERROR_NOT_SUPPORTED if present in the
fatbin, even if it is not actually executed.
@pgorlani pgorlani requested a review from bader as a code owner January 24, 2022 17:43
@pgorlani
Copy link
Contributor Author

Hi @bader, probably Jenkins/Precommit is stuck, it's still in Pending — In Progress

@bader
Copy link
Contributor

bader commented Jan 26, 2022

It's not stuck, but I think llvm-test-suite testing on Level Zero backend has been broken recently, which caused significant testing time increase.
I'm merging this PR as Jenkins/Precommit job doesn't cover modified sources.

@bader bader merged commit ec29322 into intel:sycl Jan 26, 2022
@pgorlani
Copy link
Contributor Author

Thanks!

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.

2 participants