[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:

thanks for taking the fix! we find many issues via our code.. https://github.com/mflowcode/mfc 

If you search for my GH username, there may be one or two left open as GH issues. I didn't push a fix to those to avoid being... pushy. But the community seems to have been supportive of fixing those.

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