[Mlir-commits] [mlir] [mlir][sparse] enable rt path for transpose COO (PR #76747)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Tue Jan 2 11:40:50 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir-sparse
@llvm/pr-subscribers-mlir
Author: Aart Bik (aartbik)
<details>
<summary>Changes</summary>
COO has been supported for a while now on both
lib and codegen path. So enabling this test for
all paths and removing obsolete FIXME
---
Full diff: https://github.com/llvm/llvm-project/pull/76747.diff
1 Files Affected:
- (modified) mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_transpose_coo.mlir (+1-2)
``````````diff
diff --git a/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_transpose_coo.mlir b/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_transpose_coo.mlir
index 95030fdf258bf0..dba897334830ad 100644
--- a/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_transpose_coo.mlir
+++ b/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_transpose_coo.mlir
@@ -17,8 +17,7 @@
// DEFINE: %{env} =
//--------------------------------------------------------------------------------------------------
-// FIXME: lib path does not support all of COO yet
-// R_U_N: %{compile} | %{run} | FileCheck %s
+// RUN: %{compile} | %{run} | FileCheck %s
//
// Do the same run, but now with direct IR generation.
// REDEFINE: %{sparsifier_opts} = enable-runtime-library=false
``````````
</details>
https://github.com/llvm/llvm-project/pull/76747
More information about the Mlir-commits
mailing list