[flang-commits] [flang] [flang][OpenMP] Support custom mappers in target update to/from clauses (PR #169673)

[Krish Gupta via flang-commits]

================

----------------
KrxGu wrote:

> Test is still not passing
> 
> ```
> flang -fopenmp -fopenmp-version=50 -fopenmp-targets=amdgcn-amd-amdhsa --offload-arch=gfx1100 -O3 test.f90 -o a.out && ./a.out 
> warning: OpenMP support for version 50 in flang is still incomplete
> warning: OpenMP support for version 50 in flang is still incomplete
> warning: OpenMP support for version 50 in flang is still incomplete
>  Sum of a (custom mapper): 1000
>  Sum of b (never mapped): 10100
> omptarget message: explicit extension not allowed: host address specified is 0x000064cc7ba9a660 (800 bytes), but device allocation maps to host at 0x000064cc7ba9a660 (400 bytes)
> omptarget error: Call to getTargetPointer returned null pointer (device failure or illegal mapping).
> omptarget error: Call to targetDataBegin via targetDataMapper for custom mapper failed.
> omptarget error: Consult https://openmp.llvm.org/design/Runtimes.html for debugging options.
> omptarget error: Source location information not present. Compile with -g or -gline-tables-only.
> omptarget fatal error 1: failure of target construct while offloading is mandatory
> Aborted (core dumped)
> ```
> 
> We can't land a patch without a test, even more so when the test is failing.
> 
> Unfortunately, I'm going on vacation until next year. Hopefully you can find someone else who can help you with the offloading test.
> 
> Cheers

@tblah , @kparzysz can someone please help me with this, I really want this PR to get merged.

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