jlebar accepted this revision. jlebar added a comment. This revision is now accepted and ready to land.
I know it comes in a separate change, but can we add a check to the test-suite? ================ Comment at: clang/lib/Headers/__clang_cuda_runtime_wrapper.h:381 +__device__ inline __cuda_builtin_threadIdx_t::operator dim3() const { + return {x, y, z}; +} ---------------- This is a C++11-ism (right?). Do we support compiling without C++11? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88250/new/ https://reviews.llvm.org/D88250 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits