[all-commits] [llvm/llvm-project] b6b075: [flang] Allow lowering of sub-expressions to be ov...

[jeanPerier via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b6b0756ce5c4e2e07d7f6f1f430d3d29afe9a8a8
      https://github.com/llvm/llvm-project/commit/b6b0756ce5c4e2e07d7f6f1f430d3d29afe9a8a8
  Author: jeanPerier <jperier at nvidia.com>
  Date:   2023-10-25 (Wed, 25 Oct 2023)

  Changed paths:
    M flang/include/flang/Lower/AbstractConverter.h
    M flang/lib/Lower/Bridge.cpp
    M flang/lib/Lower/ConvertExpr.cpp
    M flang/lib/Lower/ConvertExprToHLFIR.cpp
    M flang/lib/Lower/DirectivesCommon.h
    M flang/test/Lower/OpenACC/acc-atomic-capture.f90
    M flang/test/Lower/OpenACC/acc-atomic-update-hlfir.f90
    M flang/test/Lower/OpenACC/acc-atomic-update.f90
    M flang/test/Lower/OpenMP/FIR/atomic-capture.f90
    M flang/test/Lower/OpenMP/FIR/atomic-update.f90
    M flang/test/Lower/OpenMP/atomic-update-hlfir.f90
    A flang/test/Lower/OpenMP/common-atomic-lowering.f90

  Log Message:
  -----------
  [flang] Allow lowering of sub-expressions to be overridden (#69944)

OpenACC/OpenMP atomic lowering needs a finer control over expression
lowering. This patch allows mapping evaluate::Expr<T> to mlir::Value so
that any subsequent expression lowering will use these values when an
operand is a mapped Expr<T>.

This is an alternative to
https://github.com/llvm/llvm-project/pull/69866 From which I took the
test and some of the logic to extract the non-atomic sub-expression.

---------

Co-authored-by: Nimish Mishra <neelam.nimish at gmail.com>