[Openmp-commits] [PATCH] D68100: [OpenMP 5.0] declare mapper runtime implementation

[Lingda Li via Phabricator via Openmp-commits]

lildmh added a comment.

In D68100#2168350 <https://reviews.llvm.org/D68100#2168350>, @grokos wrote:

> @lildmh I've got a question unrelated to the problem we are discussing here. I ran `declare_mapper_target.cpp` and when libomptarget calls the mapper function it prints the following:
>
>   Libomptarget --> __tgt_push_mapper_component(Handle=0x00007ffcebd0fb48) adds an entry (Base=0x00007ffcebd101e0, Begin=0x00007ffcebd101e0, Size=8, Type=0x20).
>   Libomptarget --> __tgt_push_mapper_component(Handle=0x00007ffcebd0fb48) adds an entry (Base=0x00007ffcebd101e0, Begin=0x000000000231bfe0, Size=4096, Type=0x2000000000013)
>
>
> Why is the second entry's MEMBER_OF field set to 2? It should be MEMBER_OF 1, since the pointer-pointee pair `c.a[0:N]` is part of struct c which is the first entry on the list.


Good point. May be a bug. Let me check later


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68100