[Openmp-commits] [PATCH] D100600: [OpenMP] Add info for device table changes

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 added inline comments.


================
Comment at: openmp/libomptarget/src/device.cpp:267
     uintptr_t tp = (uintptr_t)allocData(Size, HstPtrBegin);
-    DP("Creating new map entry: HstBase=" DPxMOD ", HstBegin=" DPxMOD ", "
-       "HstEnd=" DPxMOD ", TgtBegin=" DPxMOD "\n",
-       DPxPTR(HstPtrBase), DPxPTR(HstPtrBegin),
-       DPxPTR((uintptr_t)HstPtrBegin + Size), DPxPTR(tp));
+    INFO(OMP_INFOTYPE_ALL, DeviceID,
+         "Creating new map entry with "
----------------
grokos wrote:
> Do you really want to use `OMP_INFOTYPE_ALL` here? Upon removal of elements, we use `OMP_INFOTYPE_CHANGED`.
Thanks for catching that, I had an if statement there before but I removed it and forgot to change that part back.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100600