[all-commits] [llvm/llvm-project] 50b907: Added RecursiveMemoryEffects to ExecuteRegionOp (...

[ddubov100 via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 50b907751f4b2b01e61e4ca25b18734a4507d8f5
      https://github.com/llvm/llvm-project/commit/50b907751f4b2b01e61e4ca25b18734a4507d8f5
  Author: ddubov100 <155631080+ddubov100 at users.noreply.github.com>
  Date:   2025-10-28 (Tue, 28 Oct 2025)

  Changed paths:
    M mlir/include/mlir/Dialect/SCF/IR/SCFOps.td
    M mlir/test/Dialect/Bufferization/Transforms/OwnershipBasedBufferDeallocation/dealloc-memoryeffect-interface.mlir
    M mlir/test/Dialect/Bufferization/Transforms/one-shot-module-bufferize.mlir

  Log Message:
  -----------
  Added RecursiveMemoryEffects to ExecuteRegionOp  (#164390)

Added RecursiveMemoryEffects to ExecuteRegionOp to be aligned to other
ops with region and get appropriate support in all appropriate passes,
which need RecursiveMemoryEffects.
The added test in dealloc-memoryeffect-interface.mlir fails with error
'ops with unknown memory side effects are not supported' without
RecursiveMemoryEffects.
The updated test in one-shot-module-bufferize.mlir gets cleaned by DCE
once the interface is added. Added func.call @foo():()->() which has
effect to keep execute_region from being removed.

---------

Co-authored-by: Mehdi Amini <joker.eph at gmail.com>



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