[Openmp-commits] [PATCH] D93505: [OpenMP] Add definition/interface for target memory routines

Hansang Bae via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Jan 4 15:36:27 PST 2021


hbae added inline comments.


================
Comment at: openmp/runtime/src/include/omp_lib.h.var:593
+          integer (kind=kmp_size_t_kind), value :: size
+          integer (kind=omp_integer_kind), value :: device_num
+        end function omp_target_alloc
----------------
kkwli0 wrote:
> Why not `c_int`?  Will it be guaranteed that this will be interoperable with `int`?
Yes, it is safer to follow 5.1 specification.
I will upload a follow-up patch soon.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93505



More information about the Openmp-commits mailing list