[Mlir-commits] [flang] [mlir] [openmp] [MLIR][OpenMP] Add scan reduction lowering to llvm (PR #167031)

[Sergio Afonso]

================

----------------
skatrak wrote:

Shouldn't this function return the closest non-parallel-related alloca insert point (i.e. `OpenMPAllocaStackFrame`) when there's no `OpenMPParallelAllocaStackFrame`? Otherwise, this would cause problems when lowering `omp.scan`:
```mlir
llvm.func ... {
  omp.wsloop reduction(...) {
    omp.loop_nest ... {
      ...
      omp.scan ...
      ...
      omp.yield
    }
  }
  return
}
```
Perhaps return the result of `findAllocaInsertPoint` if the `moduleTranslation.stackWalk` is not interrupted (i.e. it didn't find any `OpenMPParallelAllocaStackFrame`s).

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