kkwli0 added inline comments.
================ Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7046-7049 + /// Signal that the runtime library should use args as an array of + /// descriptor_dim pointers and use args_size as dims. Used when we have + /// non-contiguous list items in target update directive + OMP_MAP_DESCRIPTOR = 0x800, ---------------- ABataev wrote: > cchen wrote: > > ABataev wrote: > > > I'm not sure about the value of this flag. If I do recall it correctly, > > > this value might be used for something different by XL compiler, for > > > example. Maybe use some other value, maybe high bits? It is a kind of > > > service flag, not data mapping attribute, so better to move it to high > > > bits (the bit before OMP_MAP_MEMBER_OF maybe?). > > Hi @ABataev, is there any place I can find which value has been used for > > lower bits (like 0x800, 0x1000)? > I rather doubt. You can try to ask @kkwli0 We are using 0x800. I think your current choice should be fine. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79972/new/ https://reviews.llvm.org/D79972 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
