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

2017-01-24 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL292997: [OpenMP] Support for the num_threads-clause on 'target parallel'. (authored by arpith). Changed prior to commit: https://reviews.llvm.org/D29082?vs=85582&id=85662#toc Repository: rL LLVM htt

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

2017-01-24 Thread Alexey Bataev via Phabricator via cfe-commits
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(Nu

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

2017-01-24 Thread Arpith Jacob via Phabricator via cfe-commits
arpith-jacob created this revision. The num_threads-clause on the combined directive applies to the 'parallel' region of this construct. We modify the NumThreadsClause class to capture the clause expression within the 'target' region. The offload runtime call for 'target parallel' is changed to