[all-commits] [llvm/llvm-project] 620669: [mlir][sparse] Renaming mlir_sparse_tensor_utils l...

wren romano via All-commits all-commits at lists.llvm.org
Tue Oct 11 15:00:30 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 62066929310fc95bea1bacf8ec26acbb4f216d4c
      https://github.com/llvm/llvm-project/commit/62066929310fc95bea1bacf8ec26acbb4f216d4c
  Author: wren romano <2998727+wrengr at users.noreply.github.com>
  Date:   2022-10-11 (Tue, 11 Oct 2022)

  Changed paths:
    A mlir/include/mlir/ExecutionEngine/SparseTensorRuntime.h
    R mlir/include/mlir/ExecutionEngine/SparseTensorUtils.h
    M mlir/lib/ExecutionEngine/CMakeLists.txt
    M mlir/lib/ExecutionEngine/SparseTensor/CMakeLists.txt
    A mlir/lib/ExecutionEngine/SparseTensorRuntime.cpp
    R mlir/lib/ExecutionEngine/SparseTensorUtils.cpp
    M utils/bazel/llvm-project-overlay/mlir/BUILD.bazel

  Log Message:
  -----------
  [mlir][sparse] Renaming mlir_sparse_tensor_utils library to SparseTensorRuntime

The "mlir_xxx_utils" naming scheme is reserved/intended for shared libraries, whereas this library must be static due to issues of linking DLLs on Windows.  So we rename the library to avoid any potential confusion.   In addition we also rename the ExecutionEngine/SparseTensorUtils.{h,cpp} files to match the new library name.

Reviewed By: aartbik, stella.stamenova

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




More information about the All-commits mailing list