[Openmp-commits] [PATCH] D67326: [OpenMP] FreeBSD address check if mapped more native

[David CARLIER via Phabricator via Openmp-commits]

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


================
Comment at: openmp/runtime/src/z_Linux_util.cpp:2027
+  int mib[] = {CTL_KERN, KERN_PROC, KERN_PROC_VMMAP, getpid()};
+  rc = sysctl(mib, 4, nullptr, &lstsz, nullptr, 0);
+  if (rc < 0)
----------------
Hahnfeld wrote:
> Even though it's C++, should this pass `NULL` to the C function `sysctl`?
Oh that works with the C++ semantic but can put C's NULL no problems.


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D67326