[PATCH] D17019: [OpenMP] Code generation for teams - kernel launching

Samuel Antao via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 28 16:40:03 PST 2016


sfantao added inline comments.

================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:4002
@@ +4001,3 @@
+static llvm::Value *
+emitNumTeamsClauseForTargetDirective(CodeGenFunction &CGF,
+                                     const OMPExecutableDirective &D,
----------------
ABataev wrote:
> I still don't like the generation of some functions, that, generally speaking, are not required.
> Could you try to add a new 'class CGOpenMPInlinedRegionInfo' like class, that will be able to handle not captured variables in expressions?
Ok. In the the new diff I am using the logic for emission of inlined regions for num teams and thread limit as well. I still had to add extra logic in CodeGenFunction to insert extra entries in the local declaration cache, given that, as I discussed above, target regions make captured global variables local.  Hope this is aligned with what you have in mind.

Thanks! 


http://reviews.llvm.org/D17019





More information about the cfe-commits mailing list