[clang] [llvm] [openmp] [offload][OpenMP] Re-enable strict modifier for num_threads (PR #163565)

[Kevin Sala Penades via cfe-commits]

================
@@ -277,6 +307,23 @@ __kmpc_parallel_51(IdentTy *ident, int32_t, int32_t if_expr,
     __kmpc_end_sharing_variables();
 }
 
+[[clang::always_inline]] void
+__kmpc_parallel_51(IdentTy *ident, int32_t id, int32_t if_expr,
+                   int32_t num_threads, int proc_bind, void *fn,
+                   void *wrapper_fn, void **args, int64_t nargs) {
+  return __kmpc_parallel_impl(ident, id, if_expr, num_threads, proc_bind, fn,
+                              wrapper_fn, args, nargs);
+}
+
+[[clang::always_inline]] void
+__kmpc_parallel_60(IdentTy *ident, int32_t id, int32_t if_expr,
----------------
kevinsala wrote:

I see, seems that OpenMPOPT handles `__kmpc_parallel_51` but not `__kmpc_parallel_60`. The only difference is the strict modifier. Do you think it's ok to still use `__kmpc_parallel_51` and use the last parameter `nt_strict` with default value `false`?

https://github.com/llvm/llvm-project/pull/163565