Skip to content

[mlir][sparse] re-enable aarch64 test. #71855

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

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

PeimingLiu
Copy link
Member

Should have been fixed by initializing output tensor to zeros in #71845

@llvmbot llvmbot added mlir:sparse Sparse compiler in MLIR mlir labels Nov 9, 2023
@llvmbot
Copy link
Member

llvmbot commented Nov 9, 2023

@llvm/pr-subscribers-mlir-sparse

@llvm/pr-subscribers-mlir

Author: Peiming Liu (PeimingLiu)

Changes

Should have been fixed by initializing output tensor to zeros in #71845


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

1 Files Affected:

  • (modified) mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_block_matmul.mlir (-3)
diff --git a/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_block_matmul.mlir b/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_block_matmul.mlir
index 7e9c0ae71a7b7e6..86cda0831fe3273 100644
--- a/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_block_matmul.mlir
+++ b/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_block_matmul.mlir
@@ -17,9 +17,6 @@
 // DEFINE: %{env} =
 //--------------------------------------------------------------------------------------------------
 
-// FIXME: make aarch64 working
-// UNSUPPORTED: target={{.*aarch64.*}}
-
 // RUN: %{compile} | %{run} | FileCheck %s
 //
 // Do the same run, but now with direct IR generation.

@PeimingLiu PeimingLiu merged commit a2d9d2e into llvm:main Nov 9, 2023
@PeimingLiu PeimingLiu deleted the aarch64-enable branch November 9, 2023 19:46
zahiraam pushed a commit to zahiraam/llvm-project that referenced this pull request Nov 20, 2023
Should have been fixed by initializing output tensor to zeros in
llvm#71845
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