[Mlir-commits] [mlir] b149fc7 - [mlir][scf] Quick fix to scf.execute_region no_inline (#154931)

[llvmlistbot at llvm.org]

Author: Jungwook Park
Date: 2025-08-22T13:11:27+01:00
New Revision: b149fc7755b1ddf95060df1b87a3b0a9890a4e40

URL: https://github.com/llvm/llvm-project/commit/b149fc7755b1ddf95060df1b87a3b0a9890a4e40
DIFF: https://github.com/llvm/llvm-project/commit/b149fc7755b1ddf95060df1b87a3b0a9890a4e40.diff

LOG: [mlir][scf] Quick fix to scf.execute_region no_inline (#154931)

Asm printer should exclude `no_inline` attr during printing optional
attrs at the bottom.

Added: 
    

Modified: 
    mlir/lib/Dialect/SCF/IR/SCF.cpp
    mlir/test/Dialect/SCF/canonicalize.mlir

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/SCF/IR/SCF.cpp b/mlir/lib/Dialect/SCF/IR/SCF.cpp
index 0dbc041d231a2..84f9777a443fd 100644
--- a/mlir/lib/Dialect/SCF/IR/SCF.cpp
+++ b/mlir/lib/Dialect/SCF/IR/SCF.cpp
@@ -157,8 +157,7 @@ void ExecuteRegionOp::print(OpAsmPrinter &p) {
   p.printRegion(getRegion(),
                 /*printEntryBlockArgs=*/false,
                 /*printBlockTerminators=*/true);
-
-  p.printOptionalAttrDict((*this)->getAttrs());
+  p.printOptionalAttrDict((*this)->getAttrs(), /*elidedAttrs=*/{"no_inline"});
 }
 
 LogicalResult ExecuteRegionOp::verify() {

diff  --git a/mlir/test/Dialect/SCF/canonicalize.mlir b/mlir/test/Dialect/SCF/canonicalize.mlir
index 7804cc7026d67..2752c492cb2be 100644
--- a/mlir/test/Dialect/SCF/canonicalize.mlir
+++ b/mlir/test/Dialect/SCF/canonicalize.mlir
@@ -1479,7 +1479,7 @@ func.func @execute_region_no_inline() {
 // CHECK-NEXT:       scf.execute_region
 // CHECK-NEXT:       %[[VAL:.*]] = "test.val"() : () -> i64
 // CHECK-NEXT:       scf.yield %[[VAL]] : i64
-// CHECK-NEXT:     }
+// CHECK-NOT:      no_inline
 
 // -----