[all-commits] [llvm/llvm-project] b80508: [mlir][spirv] Introduce OwningSPIRVModuleRef for o...

[Lei Zhang via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: b80508703fd7f88a7922c9c8f02c696be1db8034
      https://github.com/llvm/llvm-project/commit/b80508703fd7f88a7922c9c8f02c696be1db8034
  Author: Lei Zhang <antiagainst at google.com>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

  Changed paths:
    A mlir/include/mlir/Dialect/SPIRV/SPIRVModule.h
    M mlir/include/mlir/Dialect/SPIRV/Serialization.h
    M mlir/include/mlir/IR/Module.h
    A mlir/include/mlir/IR/OwningOpRefBase.h
    M mlir/lib/Dialect/SPIRV/Serialization/Deserializer.cpp
    M mlir/lib/Dialect/SPIRV/Serialization/TranslateRegistration.cpp
    M mlir/unittests/Dialect/SPIRV/DeserializationTest.cpp

  Log Message:
  -----------
  [mlir][spirv] Introduce OwningSPIRVModuleRef for ownership

Similar to OwningModuleRef, OwningSPIRVModuleRef signals ownership
transfer clearly. This is useful for APIs like spirv::deserialize,
where a spirv::ModuleOp is returned by deserializing SPIR-V binary
module.

This addresses the ASAN error as reported in
https://bugs.llvm.org/show_bug.cgi?id=46272

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