[flang-commits] [flang] [llvm] [Flang][OpenMP] Fix crash in MapInfoFinalization (PR #192732)

[via flang-commits]

agozillon wrote:

I actually think I have a recent fix for this downstream (and I came across this same fix while doing so, so it seemed familiar!) that tries to support this edge-case a bit more completely (I think at least), rather than skip it, it makes sure we have the bindings setup before attempting it:  https://github.com/ROCm/llvm-project/pull/1885/ It's a bit more involved though. I was planning to open a PR for it once it settled a bit (which it has now, all green downstream, so just hadn't worked up the time for it yet) :-)

But in any case, I suppose our options are:
     1) Merge this PR and merge my PR later to incorporate an additional layer of support
     2) You can incorporate my PR into your PR so we have it all in this PR
     3) We drop my PR or your PR in place of the other

I don't mind any of the above scenarios, so let me know if you think the PRs address the same issue (just one enabling the cases and the other circumventing them/any cases not caught by the enablement) and if you do, which of the above options you'd prefer to go with .  

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