[flang-commits] [flang] [flang][OpenMP] Handle REQUIRES ADMO in lowering (PR #144362)
Kiran Chandramohan via flang-commits
flang-commits at lists.llvm.org
Wed Jun 18 02:36:59 PDT 2025
kiranchandramohan wrote:
> 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.
Feel free to go ahead if both of you are in agreement. It was requested to do this in semantics since we did not want to complicate either the lowering or the translation. And have the semantics phase enforce the semantics implied by the standard for requires.
https://github.com/llvm/llvm-project/pull/144362
More information about the flang-commits
mailing list