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

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

I think this comment could be simplified a bit, we likely don't need the performance explanation/issue link as it's a bit of extra detail, simply stating that it's unnecessary in these cases should suffice (anyone needing more detail can likely git blame and do there research based on your original commit). I think removing the / in the "allocatable/nested" components might also aid the explanation, but that might just be me!

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