[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