[flang-commits] [flang] [Flang][OpenMP] Align map clause generation and fix issue with non-shared allocations for assumed shape/size descriptor types (PR #97855)

[via flang-commits]

https://github.com/agozillon commented:

No new cases really being handled, just an overall simplification of the code and making both implicit map and explicit maps aligned. However, there's also a minor bug fix via sharing locally made temporary allocations that are load/stored into (fixed by both the OMPMapInfoFinalization pass changes and to a lesser extent by avoiding some cases with the swap to raw input, but the former should hopefully catch any cases the latter doesn't avoid). I am more than happy to add a test case for the bug fix to the runtime tests as that's primarily where the problem will be detected, as it occurs as a runtime mapping error!

I'll update the test and have a look into swapping back to a module only pass, it'll require a bit more complexity in the pass as I'll have to work out a way to appropriately track which allocations belong to which function. Likely not too difficult though! 

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