[PATCH] D110429: [OpenMP] Introduce a new worksharing RTL function for distribute

Mon Sep 27 08:08:44 PDT 2021

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

LG.

We might be able to revert/simplify this eventually again but for now it makes it much simpler to distinguish the distribute and workshare (=for) via their call site.

================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:2879
+    StaticInitFunction =
+        createForStaticInitFunction(Values.IVSize, Values.IVSigned);
+
----------------
Could we pass the 4 potential functions to `createForStaticInitFunction` (or similar name) to avoid the duplication? Or just pass a flag.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110429