[Openmp-commits] [PATCH] D56102: [OPENMP][NVPTX]Added/fixed debugging messages, NFC.

Alexey Bataev via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Dec 27 08:21:29 PST 2018


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


================
Comment at: libomptarget/deviceRTLs/nvptx/src/supporti.h:241-242
   void *ptr = malloc(size);
-  PRINT(LD_MEM, "malloc data of size %zu for %s: 0x%llx\n", size, msg,
-        (unsigned long long)ptr);
+  PRINT(LD_MEM, "malloc data of size %llu for %s: 0x%llx\n",
+        (unsigned long long)size, msg, (unsigned long long)ptr);
   return ptr;
----------------
grokos wrote:
> Why do we need to cast `size` to `unsigned long long`?
`%zu` modifier is not recognized and it leads to the crash at the runtime.


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D56102





More information about the Openmp-commits mailing list