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

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 16:17:08 PST 2022


qcolombet marked 2 inline comments as done.
qcolombet added inline comments.


================
Comment at: mlir/test/Dialect/SparseTensor/vectorize_reduction.mlir:438
+
+// Check that we don't vectorize subi when the input value is on the lhs.
+// I.e., res = res - input // Ok, tested in sparse_reduction_subi
----------------
aartbik wrote:
> can you remove this part from the test now?
Sure thing!
I’ll do that next week


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