t-tye added inline comments.
================ Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:918 + // + // ToDo: clang does not support CUDA/HIP dynamic parallelism, therefore + // CUDA/HIP kernel can be marked with norecurse. This may change in the ---------------- tra wrote: > I believe dynamic parallelism is more like spawning a separate process, not > recursion, so we should be OK. Right, dynamic parallelism is equivalent to spawn a new thread to execute the function. Recursion is only about the same thread calling the function. So I agree that the TODO about it can be removed. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73651/new/ https://reviews.llvm.org/D73651 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
