[PATCH] D110089: [CUDA] Implement experimental support for texture lookups.

Thu Oct 7 16:29:14 PDT 2021

kgk added a comment.

In D110089#3048460 <https://reviews.llvm.org/D110089#3048460>, @tra wrote:

> In D110089#3047161 <https://reviews.llvm.org/D110089#3047161>, @kgk wrote:
>
>> Will the new macros in this patch also be useful for supporting the surface-related methods that also use __nv_tex_surf_handler (from surface_indirect_functions.h)?
>>
>> I gave this new code a try with surf2Dread and surf2Dwrite, and based on the errors, it look like it may just be a matter of creating the right mappings from Tag to the correct asm using these new macros (e.g. __isurf2Dwrite_v2 and isurf2Dread).
>
> Only textures are supported at the moment, but adding support for surface operations would indeed be very similar.
>
> Basically we just need to add specializations for the surface operations. It's fairly tedious, but straightforward in principle.

Very cool! I am selfishly curious if support for surface operations is something you plan to add. I had a go at implementing it myself today based on this patch, and found it a bit harder than I was expecting 😅

I appreciate your work on this; it's great to see cuda texture support being added to clang!

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110089/new/

https://reviews.llvm.org/D110089