[all-commits] [llvm/llvm-project] 6cba96: Add some missing CMake dependencies between MLIR d...

[Mehdi Amini via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6cba96332b3ae2cbf3f9b62dfb407f3275f6cf79
      https://github.com/llvm/llvm-project/commit/6cba96332b3ae2cbf3f9b62dfb407f3275f6cf79
  Author: Mehdi Amini <joker.eph at gmail.com>
  Date:   2021-07-28 (Wed, 28 Jul 2021)

  Changed paths:
    M mlir/lib/Dialect/GPU/CMakeLists.txt
    M mlir/lib/Dialect/OpenACC/CMakeLists.txt
    M mlir/lib/Dialect/SparseTensor/IR/CMakeLists.txt

  Log Message:
  -----------
  Add some missing CMake dependencies between MLIR dialects (NFC)


  Commit: 660a56956c32b0bcd850fc12fa8ad0225a6bb880
      https://github.com/llvm/llvm-project/commit/660a56956c32b0bcd850fc12fa8ad0225a6bb880
  Author: Mehdi Amini <joker.eph at gmail.com>
  Date:   2021-07-28 (Wed, 28 Jul 2021)

  Changed paths:
    M mlir/include/mlir/Support/TypeID.h
    M mlir/tools/mlir-tblgen/AttrOrTypeDefGen.cpp
    M mlir/tools/mlir-tblgen/DialectGen.cpp
    M mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp

  Log Message:
  -----------
  Emit strong definition for TypeID storage in Op/Type/Attributes definition

By making an explicit template specialization for the TypeID provided by these classes,
the compiler will not emit an inline weak definition and rely on the linker to unique it.
Instead a single definition will be emitted in the C++ file alongside the implementation
for these classes. That will turn into a linker error what is now a hard-to-debug runtime
behavior where instances of the same class may be using a different TypeID inside of
different DSOs.

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


Compare: https://github.com/llvm/llvm-project/compare/da7a5c09c86c...660a56956c32