Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MLIR][sparse] Fix SparseTensor test_output.py test #110882

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

mtsokol
Copy link
Contributor

@mtsokol mtsokol commented Oct 2, 2024

This PR fixes a test failure introduced in #109135

@llvmbot
Copy link
Collaborator

llvmbot commented Oct 2, 2024

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-sparse

Author: Mateusz Sokół (mtsokol)

Changes

This PR fixes a test failure introduced in #109135


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

1 Files Affected:

  • (modified) mlir/test/Integration/Dialect/SparseTensor/python/test_output.py (+1-1)
diff --git a/mlir/test/Integration/Dialect/SparseTensor/python/test_output.py b/mlir/test/Integration/Dialect/SparseTensor/python/test_output.py
index 7d9aa37ba2890c..b1438bd71055d6 100644
--- a/mlir/test/Integration/Dialect/SparseTensor/python/test_output.py
+++ b/mlir/test/Integration/Dialect/SparseTensor/python/test_output.py
@@ -183,7 +183,7 @@ def main():
         build_compile_and_run_output(attr, compiler, expected(2))
         count = count + 1
 
-    # CHECK: Passed 17 tests
+    # CHECK: Passed 21 tests
     print("Passed", count, "tests")
 
 

@PeimingLiu PeimingLiu merged commit e3686f1 into llvm:main Oct 2, 2024
11 checks passed
@mtsokol mtsokol deleted the test_output-fix branch October 3, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mlir:sparse Sparse compiler in MLIR mlir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants