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

Some nits noted below. Can be done prior to commit.



================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:2374
+    break;
+  }
   case OMPRTL__kmpc_alloc: {
----------------
This reminds me, we should replace this code with a generator. I'll look into 
it.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:6885
+        llvm::ConstantInt::get(CGM.IntTy, Data.IsWorksharingReduction ? 1 : 0,
+                               /*isSigned=*/true),
+        llvm::ConstantInt::get(CGM.IntTy, Size, /*isSigned=*/true),
----------------
Nit: Unclear if the ternary operator is really needed.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.h:2246
+  /// ...
+  /// void* tg1 = __kmpc_taskred_modifier_init(loc, gtid, is_worksharing, n,
+  /// red_data);
----------------
`__kmpc_task_reduction_modifier_init`, same elsewhere.


================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.h:2260
+  /// \code
+  /// __kmpc_taskred_modifier_fini(loc, gtid, is_worksharing);
+  /// \endcode
----------------
`__kmpc_task_reduction_modifier_fini`, same elsewhere


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79034/new/

https://reviews.llvm.org/D79034



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to