[flang-commits] [flang] [llvm] [mlir] [Flang][OpenMP] Fix Fortran automap handling (PR #162501)

[via flang-commits]

================
@@ -1612,7 +1612,9 @@ class MapInfoFinalizationPass
                "single users or up to two users when those users"
                "are a MapInfoOp and Target mapping directive");
 
-        if (hasADescriptor(op.getVarPtr().getDefiningOp(),
+        if (!bitEnumContainsAll(op.getMapType(),
----------------
agozillon wrote:

Probably a silly question, but do we still need this check for the attach map flag? I don't think we apply this outside of this pass, if I remember correctly at least... which I might not after being away for a few weeks :-) Is there another case we'd need this for, do we perhaps trigger this by generating an attach map inside of here and then re-processing it? Happy to keep it if it's necessary, just would love to know the reason so I can watch out for it in the future!

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