[flang-commits] [flang] [flang][OpenMP] Common lowering flow for atomic update (PR #69866)

[via flang-commits]

https://github.com/jeanPerier commented:

Thanks for the patch, and thanks for the summary @kiranchandramohan.
I think it would still be better to add the lowering infrastructure to have a finer control over expression lowering and allow mapping  pre-lowered mlir::Value sub-expression to their evaluate::Expr<T> representation so that later lowering can use the value without lowering the sub-expression again.

I think this may have more use cases than this and will avoid duplicating expression lowering.

I made a draft proposal in https://github.com/llvm/llvm-project/pull/69944

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