[Openmp-commits] [llvm] [openmp] [Offload][OMPT] Properly implement `ompt_get_target_info` (PR #202392)

[Jan André Reuter via Openmp-commits]

================
@@ -246,8 +255,14 @@ class Interface {
   /// Used for marking end of a data operation
   void endTargetDataOperation();
 
+  /// Used for marking begin of a submit operation
+  void beginTargetSubmitOperation();
----------------
Thyre wrote:

The `data_op` operations currently use `beginTargetDataOperation` and `endTargetDataOperation` for this, where I added resetting `HostOpId`. As far as I can tell, that change should already cover setting / unsetting `HostOpId`, right?

For calls like `omp_target_memcpy`, I don't think `ompt_get_target_info` should return a true value. There's no `target[_emi]` callback that's dispatched for those runtime calls, and the specification only mentions 

> This entry point returns one if the encountering thread is in a target region and zero otherwise.

with the runtime call not counting as being in a target region as far as I can tell.


https://github.com/llvm/llvm-project/pull/202392