[all-commits] [llvm/llvm-project] 41f362: [OpenMP][OMPT] Fix reported target pointer for dat...

Michael Halkenhäuser via All-commits all-commits at lists.llvm.org
Wed Aug 16 04:48:46 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 41f3626f8b300cef24c06d9e8b7cf53029a4330a
      https://github.com/llvm/llvm-project/commit/41f3626f8b300cef24c06d9e8b7cf53029a4330a
  Author: Michael Halkenhaeuser <MichaelGerald.Halkenhauser at amd.com>
  Date:   2023-08-16 (Wed, 16 Aug 2023)

  Changed paths:
    M openmp/libomptarget/src/OmptCallback.cpp
    M openmp/libomptarget/src/OmptInterface.h
    M openmp/libomptarget/src/device.cpp
    M openmp/libomptarget/test/ompt/veccopy_disallow_both.c
    M openmp/libomptarget/test/ompt/veccopy_emi.c
    M openmp/libomptarget/test/ompt/veccopy_emi_map.c

  Log Message:
  -----------
  [OpenMP][OMPT] Fix reported target pointer for data alloc callback

This patch fixes: https://github.com/llvm/llvm-project/issues/64671
DataOp EMI callbacks would not report the correct target pointer.
This is now alleviated by passing a `void**` into the function which
emits the actual callback, then evaluating that pointer.

Note: Since this is only done after the pointer has been properly
updated, only `endpoint=2` callbacks will show a non-null value.

Reviewed By: dhruvachak, jdoerfert

Differential Revision: https://reviews.llvm.org/D157996




More information about the All-commits mailing list