[Mlir-commits] [mlir] 51eaee3 - [mlir][SparseTensor] Fix test regression

[Andrzej Warzynski]

Author: Andrzej Warzynski
Date: 2023-08-16T09:37:07Z
New Revision: 51eaee3b42f057fc2cc3fae7ec71d44fa1b82057

URL: https://github.com/llvm/llvm-project/commit/51eaee3b42f057fc2cc3fae7ec71d44fa1b82057
DIFF: https://github.com/llvm/llvm-project/commit/51eaee3b42f057fc2cc3fae7ec71d44fa1b82057.diff

LOG: [mlir][SparseTensor] Fix test regression

Fix a regression caused by https://reviews.llvm.org/D158012. Failing
bot:
  * https://lab.llvm.org/buildbot/#/builders/179/builds/7122

Note that both `RUN` lines in the affected file were previously
tested with similar configuraiton (_with_ and _without_ vectorisation).
This change restores that, though the new setting (from D158012) is
used, i.e.

  * with direct IR generation, `enable-runtime-library=true`.

This is sufficient to make the test pass and allows us to investigate
the root cause offline. Issue reported here:

  https://github.com/llvm/llvm-project/issues/64727

Added: 
    

Modified: 
    mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_pack_libgen.mlir

Removed: 
    


################################################################################
diff  --git a/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_pack_libgen.mlir b/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_pack_libgen.mlir
index 3e4194a5b478b0..99d2a46ef4ad6c 100644
--- a/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_pack_libgen.mlir
+++ b/mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_pack_libgen.mlir
@@ -20,7 +20,7 @@
 // RUN: %{compile} | %{run} | FileCheck %s
 //
 // Do the same run, but now with VLA vectorization.
-// REDEFINE: %{sparse_compiler_opts} = enable-runtime-library=false vl=4
+// REDEFINE: %{sparse_compiler_opts} = enable-runtime-library=true vl=4
 // RUN: %if mlir_arm_sve_tests %{ %{compile_sve} | %{run_sve} | FileCheck %s %}
 
 // TODO: This is considered to be a short-living tests and should be merged with sparse_pack.mlir