[OpenMP][OMPT] Fix ompt_get_task_memory implementation

[protze.joachim]

Summary

Since td_allow_completion_event is a member of the taskdata struct, not all firstprivate/shared variables are stored at the end of the task memory allocation. Simply report the whole allocation instead.
Furthermore, the function should always return 0 since in no case there is another block to report.

[hbae]

openmp/runtime/src/ompt-specific.cpp

478–479

We can remove this check since it is always greater than 0 for an explicit task.

[protze.joachim]

Remove unnecessary check

[hbae]

LGTM

openmp/runtime/src/ompt-specific.cpp

if (thread_num) {
       //        *thread_num = team->t.t_master_tid;
     }
     return info ? 2 : 0;
   }
   return 0;
 }
 
 int __ompt_get_task_memory_internal(void **addr, size_t *size, int blocknum) {
+  *size = 0;
   if (blocknum != 0)
     return 0; // support only a single block
 
   kmp_info_t *thr = ompt_get_thread();
   if (!thr)
     return 0;
 
   kmp_taskdata_t *taskdata = thr->th.th_current_task;
-  kmp_task_t *task = KMP_TASKDATA_TO_TASK(taskdata);
 
   if (taskdata->td_flags.tasktype != TASK_EXPLICIT)
     return 0; // support only explicit task
 
-  void *ret_addr;
-  int64_t ret_size = taskdata->td_size_alloc - sizeof(kmp_taskdata_t);
-
-  // kmp_task_t->data1 is an optional member
-  if (taskdata->td_flags.destructors_thunk)
-    ret_addr = &task->data1 + 1;
-  else
-    ret_addr = &task->part_id + 1;
-
-  ret_size -= (char *)(ret_addr) - (char *)(task);
-  if (ret_size < 0)
-    return 0;
-
-  *addr = ret_addr;
-  *size = (size_t)ret_size;
-  return 1;
+  *addr = taskdata;
+  *size = taskdata->td_size_alloc;
 }
 
 //----------------------------------------------------------
 // team support
 //----------------------------------------------------------
 
 void __ompt_team_assign_id(kmp_team_t *team, ompt_data_t ompt_pid) {
   team->t.ompt_team_info.parallel_data = ompt_pid;