[all-commits] [llvm/llvm-project] 795289: [mlir][acc] Fixed side effects for [first]private/...

Slava Zakharin via All-commits all-commits at lists.llvm.org
Tue Feb 10 16:14:10 PST 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7952896611fb7e5e4dade24c9fc5583c7dc42363
      https://github.com/llvm/llvm-project/commit/7952896611fb7e5e4dade24c9fc5583c7dc42363
  Author: Slava Zakharin <szakharin at nvidia.com>
  Date:   2026-02-10 (Tue, 10 Feb 2026)

  Changed paths:
    M mlir/include/mlir/Dialect/OpenACC/OpenACCCGOps.td
    M mlir/include/mlir/Dialect/OpenACC/OpenACCOps.td
    M mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp

  Log Message:
  -----------
  [mlir][acc] Fixed side effects for [first]private/reduction. (#180791)

This patch moves the definitions of memory effects for the data
entry/exit operations into C++ code. The main reason for this
is to modify the effects of [first]private and reduction
operations: they should not access `CurrentDeviceIdResource`
when they are located inside a compute construct.

The ODS to C++ migration was done with AI assistance. I reviewed
these changes and made sure it was an NFC change. After that
I modified [first]private and reduction implementations.



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