[PATCH] D138495: [openmp][mlir] Lower parallel if to new fork_call_if function.

Mon Dec 12 12:31:10 PST 2022

mstorsjo added inline comments.

================
Comment at: openmp/runtime/src/kmp.h:3904
                                  kmpc_micro microtask, ...);
+KMP_EXPORT void __kmpc_fork_call_if(ident_t *loc, kmp_int32 nargs,
+                                    kmpc_micro microtask, kmp_int32 cond,
----------------
DavidTruby wrote:
> mstorsjo wrote:
> > This new function needs to be added to the `dllexports` file to make it available and usable on Windows.
> Ok, I can add that; do you have any clue how the numbers for functions are chosen in that file? I don't really understand how dll exporting works on Windows
In general cases, it would be enough to just add the name (or mark the symbol with a `dllexport` attribute in the source code), but here in OpenMP, the exports also are given ordinal numbers (which need to be unique). I think the logic so far is to just pick the next free number.

D131830 is adding `292` and `293` (if it ends up reapplied in its current form), so I guess we could go with `294` to avoid conflicts - unless anyone of the actual OpenMP maintainers have a better suggestion?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138495