[all-commits] [llvm/llvm-project] ecab63: [MLIR][SPIRV] Refactoring serialization and deseri...

Kareem Ergawy via All-commits all-commits at lists.llvm.org
Mon Dec 14 09:28:40 PST 2020


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ecab63894bb5aebcbbe694839779f346e6fbe9e2
      https://github.com/llvm/llvm-project/commit/ecab63894bb5aebcbbe694839779f346e6fbe9e2
  Author: ergawy <kareem.ergawy at gmail.com>
  Date:   2020-12-14 (Mon, 14 Dec 2020)

  Changed paths:
    R mlir/include/mlir/Dialect/SPIRV/SPIRVBinaryUtils.h
    R mlir/include/mlir/Dialect/SPIRV/Serialization.h
    A mlir/include/mlir/Target/SPIRV/Deserialization.h
    A mlir/include/mlir/Target/SPIRV/SPIRVBinaryUtils.h
    A mlir/include/mlir/Target/SPIRV/Serialization.h
    M mlir/lib/Conversion/GPUToVulkan/ConvertGPULaunchFuncToVulkanLaunchFunc.cpp
    M mlir/lib/Dialect/SPIRV/CMakeLists.txt
    R mlir/lib/Dialect/SPIRV/Serialization/CMakeLists.txt
    R mlir/lib/Dialect/SPIRV/Serialization/Deserializer.cpp
    R mlir/lib/Dialect/SPIRV/Serialization/SPIRVBinaryUtils.cpp
    R mlir/lib/Dialect/SPIRV/Serialization/Serializer.cpp
    R mlir/lib/Dialect/SPIRV/Serialization/TranslateRegistration.cpp
    M mlir/lib/Target/CMakeLists.txt
    A mlir/lib/Target/SPIRV/Deserialization.cpp
    A mlir/lib/Target/SPIRV/SPIRVBinaryUtils.cpp
    A mlir/lib/Target/SPIRV/Serialization.cpp
    A mlir/lib/Target/SPIRV/TranslateRegistration.cpp
    M mlir/unittests/Dialect/SPIRV/CMakeLists.txt
    M mlir/unittests/Dialect/SPIRV/DeserializationTest.cpp
    M mlir/unittests/Dialect/SPIRV/SerializationTest.cpp

  Log Message:
  -----------
  [MLIR][SPIRV] Refactoring serialization and deserialization

This commit splits SPIR-V's serialization and deserialization code
into separate libraries. The motiviation being that the serializer
is used more often the deserializer and therefore lumping them
together unnecessarily increases binary size for the most common
case.

This commit also moves these libraries into the Target/ directory
to follow MLIR convention.

Reviewed By: antiagainst

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




More information about the All-commits mailing list