[llvm-branch-commits] [flang] [Flang][OpenMP][Offload] Modify MapInfoFinalization to handle attach mapping and 6.1's ref_* and attach map keywords (PR #177715)

[via llvm-branch-commits]

================
@@ -455,6 +455,12 @@ mlir::Value createParentSymAndGenIntermediateMaps(
         interimMapType &= ~mlir::omp::ClauseMapFlags::to;
         interimMapType &= ~mlir::omp::ClauseMapFlags::from;
         interimMapType &= ~mlir::omp::ClauseMapFlags::return_param;
+        // We do not want to carry over the separation of descriptor and pointer
+        // mapping of any intermediate components we emit maps for as this can
+        // result in very odd differing behaviour when either ref_ptr/ptee is
+        // specified.
+        interimMapType &= ~mlir::omp::ClauseMapFlags::ref_ptr;
----------------
agozillon wrote:

We no longer have the ref_ptr_ptee flag, it's just the ref_ptr and ref_ptee flag at request of Michael Klemm! It's just ref_ptr and ref_ptee going upstream, so I'll have a lot of fun consolidating that difference downstream when this lands unfortunately :-)

When this all lands ref_ptr + ref_ptee on the same map will equal the old ref_ptr_ptee! Hopefully that clears that up a bit.

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