Skip to content

Conversation

@ryanpholt
Copy link
Member

Test seems to pass after re-enabling without any additional changes.

Test seems to pass after re-enabling without any additional changes.
@ryanpholt ryanpholt marked this pull request as ready for review December 1, 2025 13:40
@llvmbot
Copy link
Member

llvmbot commented Dec 1, 2025

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-linalg

Author: Ryan Holt (ryanpholt)

Changes

Test seems to pass after re-enabling without any additional changes.


Full diff: https://github.com/llvm/llvm-project/pull/170129.diff

1 Files Affected:

  • (modified) mlir/test/Integration/Dialect/Linalg/CPU/runtime-verification.mlir (+4-4)
diff --git a/mlir/test/Integration/Dialect/Linalg/CPU/runtime-verification.mlir b/mlir/test/Integration/Dialect/Linalg/CPU/runtime-verification.mlir
index 610ed63168d87..c90476e1ff61d 100644
--- a/mlir/test/Integration/Dialect/Linalg/CPU/runtime-verification.mlir
+++ b/mlir/test/Integration/Dialect/Linalg/CPU/runtime-verification.mlir
@@ -80,10 +80,10 @@ func.func @main() {
   %c64x57 = arith.constant dense<0.0> : tensor<16x29xf32>
   %c3x4 = arith.constant dense<0.0> : tensor<3x4xf32>
 
-  // TODO: BROKEN CHK: ERROR: Runtime op verification failed
-  // TODO: BROKEN CHK-NEXT: linalg.generic
-  // TODO: BROKEN CHK-NEXT: unexpected negative result on dimension #0 of input/output operand #0
-  // TODO: BROKEN func.call @reverse_from_3(%d5x) : (tensor<?xf32>) -> (tensor<?xf32>)
+  // CHECK: ERROR: Runtime op verification failed
+  // CHECK-NEXT: linalg.generic
+  // CHECK-NEXT: unexpected negative result on dimension #0 of input/output operand #0
+  func.call @reverse_from_3(%d5x) : (tensor<?xf32>) -> (tensor<?xf32>)
 
   %c0x = arith.constant dense<1.0> : tensor<0xf32>
   %d0x = tensor.cast %c0x : tensor<0xf32> to tensor<?xf32>

@ryanpholt ryanpholt merged commit b27301f into llvm:main Dec 1, 2025
15 checks passed
augusto2112 pushed a commit to augusto2112/llvm-project that referenced this pull request Dec 3, 2025
Test seems to pass after re-enabling without any additional changes.
kcloudy0717 pushed a commit to kcloudy0717/llvm-project that referenced this pull request Dec 4, 2025
Test seems to pass after re-enabling without any additional changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants