[all-commits] [llvm/llvm-project] bf6477: [MLIR][OpenMP] Place alloca scope within wsloop in...

[William Moses via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bf6477ebebf82c4c914a116a3d1d673da6d2164e
      https://github.com/llvm/llvm-project/commit/bf6477ebebf82c4c914a116a3d1d673da6d2164e
  Author: William S. Moses <gh at wsmoses.com>
  Date:   2022-03-02 (Wed, 02 Mar 2022)

  Changed paths:
    M mlir/lib/Conversion/SCFToOpenMP/SCFToOpenMP.cpp
    M mlir/test/Conversion/SCFToOpenMP/reductions.mlir
    M mlir/test/Conversion/SCFToOpenMP/scf-to-openmp.mlir

  Log Message:
  -----------
  [MLIR][OpenMP] Place alloca scope within wsloop in scf.parallel to omp lowering

https://reviews.llvm.org/D120423 replaced the use of stacksave/restore with memref.alloca_scope, but kept the save/restore at the same location. This PR places the allocation scope within the wsloop, thus keeping the same allocation scope as the original scf.parallel (e.g. no longer over stack allocating).

Reviewed By: ftynse

Differential Revision: https://reviews.llvm.org/D120772