[PATCH] D139625: [mlir][SparseTensor] Add a few tests for sparse vectorization

Aart Bik via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 8 10:16:59 PST 2022


aartbik added inline comments.


================
Comment at: mlir/test/Dialect/SparseTensor/vectorize_reduction.mlir:2
+// RUN: mlir-opt %s -sparsification -cse -sparse-vectorization="vl=8" -cse -split-input-file | \
+// RUN:   FileCheck %s --check-prefix=ON
+// RUN: mlir-opt %s -sparsification -cse -split-input-file | \
----------------
can we make this CHECK-ON/CHECK-OFF to make it more consistent with style in this dir?


================
Comment at: mlir/test/Dialect/SparseTensor/vectorize_reduction.mlir:363
+
+// Check that wer vectorize reduction with subi.
+//
----------------
wer -> we


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139625/new/

https://reviews.llvm.org/D139625



More information about the llvm-commits mailing list