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

[Krzysztof Parzyszek via flang-commits]

kparzysz wrote:

There was no single compelling reason for this.  The idea was really to implement this functionality in the same way as we do everything else---check for errors in semantic analysis, generate corresponding MLIR in lowering, etc.  Having an AST-rewriting function seemed a bit out of place.  Initially I thought it wasn't possible to do it afterwards, but it turned out to be doable without any issues.  It was sort of a logical continuation of my work on overhauling the atomic implementation, and it started when it turned out that the rules about REQUIRES ADMO have changed a bit (acquire and release were added as possible values).  To implement that we'd have to add checks to this extra location, and it seemed it would have been nice to have it all in the already known places.

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