[flang-commits] [flang] [flang][OpenMP] Don't emit implicit default mapper for flat allocatable derived types (PR #209645)

[Spencer Bryngelson via flang-commits]

================
@@ -3723,7 +3723,20 @@ genTargetOp(lower::AbstractConverter &converter, lower::SymMap &symTable,
             // Avoid attaching implicit default mappers to pointer captures.
             // For large pointer-based derived aggregates this can over-map
             // nested payloads and conflict with explicit enter/exit maps.
-            if (!isPointer && (hasDefaultMapper || isAllocatable)) {
+            //
+            // For an allocatable capture, only synthesize an implicit default
----------------
sbryngelson wrote:

agreed on all fronts. I propagated that and similar changes across the PR surface. thanks for keeping my slop in check :/ 

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