[PATCH] D79739: Adding Extra Runtime Functions to OMPKinds.def

Tue May 12 07:30:00 PDT 2020

jhuber6 marked an inline comment as done.
jhuber6 added inline comments.

================
Comment at: llvm/include/llvm/Frontend/OpenMP/OMPKinds.def:431
+__OMP_RTL(__kmpc_omp_task_alloc, false, Int8Ptr, IdentPtr, Int32, Int32, Int64,
+          Int64, TaskRoutineEntryPtr)
+__OMP_RTL(__kmpc_omp_task, false, Int32, IdentPtr, Int32, Int8Ptr)
----------------
Functions like this one use "size_t" which would be a 32 bit integer on a 32 bit system, but I've just set it to 64 bits for now. Should I add some macro's to set it to an Int32 or Int64 depending on the target? The tests target a 64 bit system and I can't see many people using OpenMP on a 32 bit computer so I wasn't sure if I should bother.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79739