[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