[all-commits] [llvm/llvm-project] b04b75: [mlir][sparse] Rename the public SparseTensorStora...

wren romano via All-commits all-commits at lists.llvm.org
Tue Aug 31 15:44:51 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b04b757a8efcb194f2a3838819e9c370f71199c8
      https://github.com/llvm/llvm-project/commit/b04b757a8efcb194f2a3838819e9c370f71199c8
  Author: wren romano <2998727+wrengr at users.noreply.github.com>
  Date:   2021-08-31 (Tue, 31 Aug 2021)

  Changed paths:
    M mlir/lib/ExecutionEngine/SparseUtils.cpp

  Log Message:
  -----------
  [mlir][sparse] Rename the public SparseTensorStorage::asCOO to toCOO

Trying to reduce confusion by having the name of the public method match that of the private method for handling the recursion.  Also adding some comments to SparseTensorStorage::fromCOO to help clarify what the recursive calls are doing in the dense case.

Reviewed By: aartbik

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




More information about the All-commits mailing list