[PATCH] [CUDA] Support for built-in cuda variables (threadIdx and its friends).

Artem Belevich tra at google.com
Fri Apr 17 16:46:32 PDT 2015


================
Comment at: lib/Headers/cuda/cuda_builtin_vars.h:3
@@ +2,3 @@
+#define __CUDA_BUILTIN_VARS_H__
+
+#define __CUDA_DEVICE_BUILTIN(FIELD, INTRINSIC)                                \
----------------
eliben wrote:
> A top-level comment on the file that explains what this does with a usage example?
Done.

================
Comment at: lib/Headers/cuda/cuda_builtin_vars.h:12
@@ +11,3 @@
+struct __cuda_builtin_threadIdx_t {
+  __CUDA_DEVICE_BUILTIN(x,__builtin_ptx_read_tid_x());
+  __CUDA_DEVICE_BUILTIN(y,__builtin_ptx_read_tid_y());
----------------
eliben wrote:
> Arent blockIdx and threadIdx unsigned?
Yes. So are blockDim and gridDim as dim3 is based on uint3. Fixed.

http://reviews.llvm.org/D9064

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list