[all-commits] [llvm/llvm-project] 8413c9: [mlir][OpenMP] initialize (first)private variables...

Tom Eccles via All-commits all-commits at lists.llvm.org
Mon Feb 24 05:04:48 PST 2025


  Branch: refs/heads/users/tblah/delayed-task-exec-4
  Home:   https://github.com/llvm/llvm-project
  Commit: 8413c94d3bcbab8912c6b17eb3647e63d92494e9
      https://github.com/llvm/llvm-project/commit/8413c94d3bcbab8912c6b17eb3647e63d92494e9
  Author: Tom Eccles <tom.eccles at arm.com>
  Date:   2025-02-24 (Mon, 24 Feb 2025)

  Changed paths:
    M mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
    M mlir/test/Target/LLVMIR/openmp-llvm.mlir

  Log Message:
  -----------
  [mlir][OpenMP] initialize (first)private variables before task exec

This still doesn't fix the memory safety issues because the stack
allocations created here for the private variables might go out of
scope.

I will add a more complete lit test later in this patch series.


  Commit: f8b1812d03f04cd8efa2a02202849e08320fa524
      https://github.com/llvm/llvm-project/commit/f8b1812d03f04cd8efa2a02202849e08320fa524
  Author: Tom Eccles <tom.eccles at arm.com>
  Date:   2025-02-24 (Mon, 24 Feb 2025)

  Changed paths:
    M mlir/test/Target/LLVMIR/openmp-llvm.mlir

  Log Message:
  -----------
  Check block structure of omp private inlining


Compare: https://github.com/llvm/llvm-project/compare/43e9c4a9cefe...f8b1812d03f0

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


More information about the All-commits mailing list