protze.joachim accepted this revision. protze.joachim added a comment. This revision is now accepted and ready to land.
The latest changes fix the issue. I tested with x86_64 and nvidia offloading. The behavior and tests looks good to me, just one comment. ================ Comment at: openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_complex_structure.cpp:123-126 + assert(outer[1].arr[1].arr[0].data1 == 11 && + outer[1].arr[1].arr[0].data2 == 22 && + outer[1].arr[1].arr[1].data1 == 11 && + outer[1].arr[1].arr[1].data2 == 22); ---------------- Do these asserts make sense with the assert in line 104? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100673/new/ https://reviews.llvm.org/D100673 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits