[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 =
+        llvm::FunctionType::get(llvm::Type::getInt1Ty(CGM.getLLVMContext()),
+                                TypeParams, /*isVarArg*/ false);
ABataev wrote:
> 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 mailing list