[flang-commits] [flang] [flang][OpenMP] Handle REQUIRES ADMO in lowering (PR #144362)

[Sergio Afonso via flang-commits]

skatrak wrote:

Thanks Krzysztof, does this address any current issues or enables adding some new feature? Or is the goal of this patch to just simplify the implementation?

I can't remember very well the details of what my initial proposal was for this or the reasoning behind what we ended up implementing, but I remember @kiranchandramohan suggesting the pass approach ([D147219](https://reviews.llvm.org/D147219#inline-1446294)).

I don't have a problem with this change, just wanted to make sure we're not missing anything by changing that approach, since I can't remember why this was originally decided.

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