[PATCH] D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device.
Arpith Jacob via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 9 04:09:15 PST 2017
arpith-jacob marked 2 inline comments as done.
arpith-jacob added inline comments.
Comment at: lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp:365
+ llvm::FunctionType *FnTy =
+ TypeParams, /*isVarArg*/ false);
> Does it really return I1 type? Or I8?
Alexey, the runtime function is called by every worker thread. It returns a 'bool' that indicates if the thread has been activated in the parallel region, which is why I used an Int1Ty. Please let me know if you see problems with this. Thanks.
More information about the cfe-commits