Skip to content

Conversation

@nyalloc
Copy link
Contributor

@nyalloc nyalloc commented Jun 24, 2020

Provides a test for functionality introduced in #1945. Should be merged after #1945.

Copy link
Contributor

@Fznamznon Fznamznon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it still WIP?

@@ -0,0 +1,15 @@
// RUN: %clang_cc1 %s -triple nvptx64-nvidia-cuda -O0 -emit-llvm -o - | FileCheck %s
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that CodeGenOpenCL test is a right place for such test...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I don't understand how you addressed @Fznamznon's question.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I don't understand how you addressed @Fznamznon's question.

No worries. It was only marked WIP because it wasn't passing the clang format check. I've resolved that now and removed the WIP status.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that CodeGenOpenCL test is a right place for such test...

I think @Naghasan can provide some insight into this.

@bader bader added the cuda CUDA back-end label Jun 24, 2020
@nyalloc nyalloc changed the title [SYCL][CUDA][WIP] Implemented sampled_image_cuda.cl [SYCL][CUDA] Implemented sampled_image_cuda.cl Jun 24, 2020
@nyalloc nyalloc closed this Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda CUDA back-end

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants