[all-commits] [llvm/llvm-project] 06aa6e: [mlir][sparse] refactor handling of merger leafs a...

[Aart Bik via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 06aa6ec87dba8ee34e4ea4bd47e6c9f2c06ebd7c
      https://github.com/llvm/llvm-project/commit/06aa6ec87dba8ee34e4ea4bd47e6c9f2c06ebd7c
  Author: Aart Bik <ajcbik at google.com>
  Date:   2022-06-09 (Thu, 09 Jun 2022)

  Changed paths:
    M mlir/lib/Dialect/SparseTensor/Utils/Merger.cpp
    M mlir/unittests/Dialect/SparseTensor/MergerTest.cpp

  Log Message:
  -----------
  [mlir][sparse] refactor handling of merger leafs and ops

Using "default:" in the switch statemements that handle all our
merger ops has become a bit cumbersome since it is easy to overlook
parts of the code that need to handle ops specifically. By enforcing
full switch statements without "default:", we get a compiler warning
when cases are overlooked.

Reviewed By: wrengr

Differential Revision: https://reviews.llvm.org/D127263