[all-commits] [llvm/llvm-project] 0d0cff: [mlir][sparse] Add Merger unit tests
Gus Smith via All-commits
all-commits at lists.llvm.org
Thu Jul 8 16:10:48 PDT 2021
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 0d0cff3ace39378acfc66d6564dc99e19b8a561f
https://github.com/llvm/llvm-project/commit/0d0cff3ace39378acfc66d6564dc99e19b8a561f
Author: Gus Smith <gusss at google.com>
Date: 2021-07-08 (Thu, 08 Jul 2021)
Changed paths:
M mlir/unittests/Dialect/CMakeLists.txt
A mlir/unittests/Dialect/SparseTensor/CMakeLists.txt
A mlir/unittests/Dialect/SparseTensor/MergerTest.cpp
Log Message:
-----------
[mlir][sparse] Add Merger unit tests
We opt to use unit tests rather than check tests as the lattice/merger code is a small C++ component with a well-defined API. Testing this API via check tests would be far less direct and readable. In addition, as the check tests will only be able to test the API indirectly, the tests may break based on unrelated changes; e.g. changes in linalg.
Reviewed By: aartbik
Differential Revision: https://reviews.llvm.org/D104956
More information about the All-commits
mailing list