hliao added a comment.

In D76365#1946345 <https://reviews.llvm.org/D76365#1946345>, @tra wrote:

> Looks like the change breaks compilation for us:
>
>   In file included from <built-in>:1:
>   In file included from 
> llvm_unstable/toolchain/lib/clang/google3-trunk/include/__clang_cuda_runtime_wrapper.h:104:
>   In file included from cuda/include/cuda_runtime.h:116: 
> cuda/include/cuda_surface_types.h:91:42: error: illegal device builtin 
> surface reference type 'surface<void, dim>' declared here
>   struct  __device_builtin_surface_type__  surface<void, dim> : public 
> surfaceReference
>                                            ^
>   cuda/include/cuda_surface_types.h:91:42: note: 'surface<void, dim>' needs 
> to be instantiated from a class template with the 2nd template argument as an 
> integral value
>   1 error generated when compiling for sm_60.
>
>
> I'm investigating, but we may need to roll back this patch. Stay tuned.


I am looking into it as well. Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76365



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to