[Openmp-commits] [PATCH] D105121: [OpenMP] Avoid checking parent reference count in targetDataBegin

[Narayanaswamy, Ravi via Openmp-commits]

If the  current behavior  is only the pointer field of the struct and not other members of the struct are updated after the array  is allocated and copied.

-----Original Message-----
From: George Rokos via Phabricator <reviews at reviews.llvm.org> 
Sent: Friday, July 09, 2021 3:41 PM
To: jdenny.ornl at gmail.com; Rokos, Georgios <georgios.rokos at intel.com>; Narayanaswamy, Ravi <ravi.narayanaswamy at intel.com>; jdoerfert at anl.gov; jhuber6 at vols.utk.edu
Cc: xw111luoye at gmail.com; sourabhsingh.tomar at amd.com; sara.royuela at bsc.es; Saiyedul.Islam at amd.com; ron.lieberman at amd.com; pdhaliwal at amd.com; lin32 at llnl.gov; liao6 at llnl.gov; kuhnel at google.com; jatin.bhateja at gmail.com; greg63706 at gmail.com; deepak.eachempati <deepak.eachempati at hpe.com>; bhuvanendra.kumarn at amd.com; abidmuslim at gmail.com; a.bataev at hotmail.com; tianshilei1992 at gmail.com; clementval at gmail.com; protze.joachim at gmail.com; georgakoudis1 at llnl.gov; yaxun.liu at amd.com; zhang.guansong at gmail.com; stefomeister at gmail.com; openmp-commits at lists.llvm.org
Subject: [PATCH] D105121: [OpenMP] Avoid checking parent reference count in targetDataBegin

grokos added a comment.

So the expected behavior is what libomptarget currently does. These 2 patches do not change it, so they are good to go.


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

https://reviews.llvm.org/D105121