doru1004 added inline comments.
================ Comment at: clang/lib/CodeGen/CGDecl.cpp:1603 + // deallocation call of __kmpc_free_shared() is emitted later. + if (getLangOpts().OpenMP && getTarget().getTriple().isAMDGCN()) { + // Emit call to __kmpc_alloc_shared() instead of the alloca. ---------------- jhuber6 wrote: > ABataev wrote: > > OpenMPIsDevice? > Does NVPTX handle this already? If not, is there a compelling reason to > exclude NVPTX? Otherwise we should check if we are the OpenMP device. Does NVPTX support dynamic allocas? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153883/new/ https://reviews.llvm.org/D153883 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits