[Mlir-commits] [mlir] 88b7915 - [mlir][linalg][test] Fix flaky test linalg-morph-multi-step.mlir (#152805)

[llvmlistbot at llvm.org]

Author: Jordan Rupprecht
Date: 2025-08-08T16:56:48-05:00
New Revision: 88b791593941e1a94499e58cb73a080b928c46d1

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

LOG: [mlir][linalg][test] Fix flaky test linalg-morph-multi-step.mlir (#152805)

This test runs `mlir-opt %s | mlir-opt %s | FileCheck` to test the round
trip behavior, but the second command takes input from the pipe, not the
lit test, so it should be `mlir-opt %s | mlir-opt | FileCheck`.

For some reason I haven't figured out, this causes ~50% flakiness when
testing in certain environments (not reproducible in my shell, but
reproduces in an internal buildbot), due to the pipeline raising
`SIGPIPE`.

Test added in #148424.

Added: 
    

Modified: 
    mlir/test/Dialect/Linalg/linalg-morph-multi-step.mlir

Removed: 
    


################################################################################
diff  --git a/mlir/test/Dialect/Linalg/linalg-morph-multi-step.mlir b/mlir/test/Dialect/Linalg/linalg-morph-multi-step.mlir
index ab50a44a37067..bdd29b96346e1 100644
--- a/mlir/test/Dialect/Linalg/linalg-morph-multi-step.mlir
+++ b/mlir/test/Dialect/Linalg/linalg-morph-multi-step.mlir
@@ -1,5 +1,5 @@
 // RUN: mlir-opt %s -linalg-morph-ops=named-to-generic |  FileCheck %s  --check-prefix=NAMED_TO_GENERIC
-// RUN: mlir-opt %s -linalg-morph-ops=named-to-generic |  mlir-opt %s -linalg-morph-ops=generic-to-named | \
+// RUN: mlir-opt %s -linalg-morph-ops=named-to-generic |  mlir-opt -linalg-morph-ops=generic-to-named | \
 // RUN:   FileCheck %s  --check-prefix=ROUND_TRIP
 
 func.func @exp(%A : tensor<16x8xf32>, %B : tensor<16x8xf32>) ->  tensor<16x8xf32> {