[PATCH] D29082: [OpenMP] Support for the num_threads-clause on 'target parallel'.

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 24 09:05:48 PST 2017


ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.

LG with a nit



================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:4996
+      CodeGenFunction::RunCleanupsScope NumThreadsScope(CGF);
+      auto NumThreads = CGF.EmitScalarExpr(NumThreadsClause->getNumThreads(),
+                                           /*IgnoreResultAssign*/ true);
----------------
Specify `llvm::Value *` instead of `auto` here.


https://reviews.llvm.org/D29082





More information about the cfe-commits mailing list