[PATCH] D88250: [CUDA] Added dim3/uint3 conversion functions to builtin vars.

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 24 11:55:03 PDT 2020


tra marked an inline comment as done.
tra added a comment.

In D88250#2293270 <https://reviews.llvm.org/D88250#2293270>, @jlebar wrote:

> I know it comes in a separate change, but can we add a check to the test-suite?

Will do.



================
Comment at: clang/lib/Headers/__clang_cuda_runtime_wrapper.h:381
+__device__ inline __cuda_builtin_threadIdx_t::operator dim3() const {
+  return {x, y, z};
+}
----------------
jlebar wrote:
> This is a C++11-ism (right?).  Do we support compiling without C++11?
Fixed. 



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88250



More information about the cfe-commits mailing list