[all-commits] [llvm/llvm-project] 0e70e0: [reapply (#118463)][OpenMP][OMPIRBuilder] Add dela...

[Kareem Ergawy via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0e70e0edd52d60bd0c063f1413b50f85ad1f4805
      https://github.com/llvm/llvm-project/commit/0e70e0edd52d60bd0c063f1413b50f85ad1f4805
  Author: Kareem Ergawy <kareem.ergawy at amd.com>
  Date:   2024-12-09 (Mon, 09 Dec 2024)

  Changed paths:
    M mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
    M mlir/test/Target/LLVMIR/openmp-todo.mlir
    A mlir/test/Target/LLVMIR/openmp-wsloop-private-cond_br.mlir
    A mlir/test/Target/LLVMIR/openmp-wsloop-private.mlir

  Log Message:
  -----------
  [reapply (#118463)][OpenMP][OMPIRBuilder] Add delayed privatization support for `wsloop` (#119170)

This reapplies PR #118463 after introducing a fix for a bug uncovere by
the test suite. The problem is that when the alloca block is terminated
with a conditional branch, this violates a pre-condition of
`allocatePrivateVars` (which assumes the alloca block has a single
successor). This new PR includes a test that reproduces the issue.

Extend MLIR to LLVM lowering by adding support for `omp.wsloop` for
delayed privatization. This also refactors a few bit of code to isolate
the logic needed for `firstprivate` initialization in a shared util that
can be used across constructs that need it. The same is done for
`dealloc` regions.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications