Skip to content

Commit 7179ffa

Browse files
committed
undo transform extension stuff
1 parent e85d9c8 commit 7179ffa

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

61 files changed

+215
-529
lines changed

mlir/cmake/modules/AddMLIRPython.cmake

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -457,14 +457,11 @@ endfunction()
457457
# This file is where the *Attrs are defined, not where the *Enums are defined.
458458
# **WARNING**: This arg will shortly be removed when the TODO for
459459
# declare_mlir_dialect_python_bindings is satisfied. Use at your risk.
460-
# EMBED_CAPI_LINK_LIBS: Dependent CAPI libraries that this extension depends
461-
# on. These will be collected for all extensions and put into an
462-
# aggregate dylib that is linked against.
463460
function(declare_mlir_dialect_extension_python_bindings)
464461
cmake_parse_arguments(ARG
465462
"GEN_ENUM_BINDINGS"
466463
"ROOT_DIR;ADD_TO_PARENT;TD_FILE;DIALECT_NAME;EXTENSION_NAME"
467-
"SOURCES;SOURCES_GLOB;DEPENDS;GEN_ENUM_BINDINGS_TD_FILE;EMBED_CAPI_LINK_LIBS"
464+
"SOURCES;SOURCES_GLOB;DEPENDS;GEN_ENUM_BINDINGS_TD_FILE"
468465
${ARGN})
469466
# Source files.
470467
set(_extension_target "${ARG_ADD_TO_PARENT}.${ARG_EXTENSION_NAME}")
@@ -506,7 +503,6 @@ function(declare_mlir_dialect_extension_python_bindings)
506503
ROOT_DIR "${CMAKE_CURRENT_BINARY_DIR}"
507504
ADD_TO_PARENT "${_extension_target}"
508505
SOURCES ${_sources}
509-
EMBED_CAPI_LINK_LIBS "${ARG_EMBED_CAPI_LINK_LIBS}"
510506
)
511507
endif()
512508
endfunction()

mlir/examples/standalone/pyproject.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ cmake.source-dir = "."
3737
# This is for installing/distributing the python bindings target and only the python bindings target.
3838
build.targets = ["StandalonePythonModules"]
3939
install.components = ["StandalonePythonModules"]
40+
# The default is true but make it explicit to highlight that this option exists (turn off for debug symbols).
41+
install.strip = true
4042

4143
[tool.scikit-build.cmake.define]
4244
# Optional
@@ -51,6 +53,9 @@ LLVM_USE_LINKER = { env = "LLVM_USE_LINKER", default = "" }
5153
CMAKE_VISIBILITY_INLINES_HIDDEN = "ON"
5254
CMAKE_C_VISIBILITY_PRESET = "hidden"
5355
CMAKE_CXX_VISIBILITY_PRESET = "hidden"
56+
# Disables generation of "version soname" (i.e. libFoo.so.<version>),
57+
# which causes pure duplication of various shlibs for Python wheels.
58+
CMAKE_PLATFORM_NO_VERSIONED_SONAME = "ON"
5459

5560
# Non-optional (alternatively you could use CMAKE_PREFIX_PATH here).
5661
MLIR_DIR = { env = "MLIR_DIR", default = "" }

mlir/examples/standalone/python/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,7 @@ set(_declared_sources
138138
StandalonePythonSources
139139
MLIRPythonSources.Core
140140
MLIRPythonSources.Dialects.builtin
141+
MLIRPythonSources.Dialects.arith
141142
)
142143
# For an external projects build, the MLIRPythonExtension.Core.type_stub_gen
143144
# target already exists and can just be added to DECLARED_SOURCES.

mlir/include/mlir-c/Dialect/Transform.h

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -76,19 +76,6 @@ MLIR_CAPI_EXPORTED MlirType mlirTransformParamTypeGet(MlirContext ctx,
7676

7777
MLIR_CAPI_EXPORTED MlirType mlirTransformParamTypeGetType(MlirType type);
7878

79-
//===---------------------------------------------------------------------===//
80-
// Extension Registration
81-
//===---------------------------------------------------------------------===//
82-
83-
MLIR_CAPI_EXPORTED void
84-
mlirDebugRegisterTransformDialectExtension(MlirDialectRegistry registry);
85-
86-
MLIR_CAPI_EXPORTED void
87-
mlirLoopRegisterTransformDialectExtension(MlirDialectRegistry registry);
88-
89-
MLIR_CAPI_EXPORTED void
90-
mlirTuneRegisterTransformDialectExtension(MlirDialectRegistry registry);
91-
9279
#ifdef __cplusplus
9380
}
9481
#endif

mlir/include/mlir-c/RegisterAllExtensions.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@
77
//
88
//===----------------------------------------------------------------------===//
99

10-
#ifndef MLIR_C_REGISTER_Extensions_H
11-
#define MLIR_C_REGISTER_Extensions_H
10+
#ifndef MLIR_C_REGISTER_EXTENSIONS_H
11+
#define MLIR_C_REGISTER_EXTENSIONS_H
1212

1313
#include "mlir-c/IR.h"
1414

@@ -23,4 +23,4 @@ MLIR_CAPI_EXPORTED void mlirRegisterAllExtensions(MlirDialectRegistry registry);
2323
}
2424
#endif
2525

26-
#endif // MLIR_C_REGISTER_Extensions_H
26+
#endif // MLIR_C_REGISTER_EXTENSIONS_H
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
//===-- mlir-c/RegisterAllExternalModels.h - Register all MLIR model ----*-===//
2+
//
3+
// Part of the LLVM Project, under the Apache License v2.0 with LLVM
4+
// Exceptions.
5+
// See https://llvm.org/LICENSE.txt for license information.
6+
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
7+
//
8+
//===----------------------------------------------------------------------===//
9+
10+
#ifndef MLIR_C_REGISTER_EXTERNAL_MODELS_H
11+
#define MLIR_C_REGISTER_EXTERNAL_MODELS_H
12+
13+
#include "mlir-c/IR.h"
14+
15+
#ifdef __cplusplus
16+
extern "C" {
17+
#endif
18+
19+
/// Register all compiler External models of MLIR.
20+
MLIR_CAPI_EXPORTED void
21+
mlirRegisterAllExternalModels(MlirDialectRegistry registry);
22+
23+
#ifdef __cplusplus
24+
}
25+
#endif
26+
27+
#endif // MLIR_C_REGISTER_EXTERNAL_MODELS_H

mlir/include/mlir-c/RegisterEverything.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ MLIR_CAPI_EXPORTED void mlirRegisterAllDialects(MlirDialectRegistry registry);
2828
/// Register all translations to LLVM IR for dialects that can support it.
2929
MLIR_CAPI_EXPORTED void mlirRegisterAllLLVMTranslations(MlirContext context);
3030

31-
// /// Register all compiler passes of MLIR.
32-
// MLIR_CAPI_EXPORTED void mlirRegisterAllPasses(void);
31+
/// Register all compiler passes of MLIR.
32+
MLIR_CAPI_EXPORTED void mlirRegisterAllPasses(void);
3333

3434
#ifdef __cplusplus
3535
}

mlir/lib/Bindings/Python/IRCore.cpp

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3139,6 +3139,10 @@ void mlir::python::populateIRCore(nb::module_ &m) {
31393139
//----------------------------------------------------------------------------
31403140

31413141
nb::class_<MlirDialectHandle>(m, "DialectHandle")
3142+
.def_prop_ro_static(
3143+
"_capsule_name",
3144+
[](nb::handle &) { return MLIR_PYTHON_CAPSULE_DIALECT_HANDLE; },
3145+
nb::sig("def _capsule_name(/) -> str"))
31423146
.def_static(MLIR_PYTHON_CAPI_FACTORY_ATTR,
31433147
&createMlirDialectHandleFromCapsule);
31443148

@@ -3147,6 +3151,10 @@ void mlir::python::populateIRCore(nb::module_ &m) {
31473151
//----------------------------------------------------------------------------
31483152
nb::class_<PyDialectRegistry>(m, "DialectRegistry")
31493153
.def_prop_ro(MLIR_PYTHON_CAPI_PTR_ATTR, &PyDialectRegistry::getCapsule)
3154+
.def_prop_ro_static(
3155+
"_capsule_name",
3156+
[](nb::handle &) { return MLIR_PYTHON_CAPSULE_DIALECT_REGISTRY; },
3157+
nb::sig("def _capsule_name(/) -> str"))
31503158
.def_static(MLIR_PYTHON_CAPI_FACTORY_ATTR,
31513159
&PyDialectRegistry::createFromCapsule)
31523160
.def(nb::init<>())

mlir/lib/Bindings/Python/RegisterEverything.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
//===----------------------------------------------------------------------===//
88

99
#include "mlir-c/RegisterAllExtensions.h"
10+
#include "mlir-c/RegisterAllExternalModels.h"
1011
#include "mlir-c/RegisterAllLLVMTranslations.h"
1112
#include "mlir-c/RegisterAllPasses.h"
1213
#include "mlir/Bindings/Python/Nanobind.h"
@@ -16,6 +17,9 @@ NB_MODULE(_mlirRegisterEverything, m) {
1617
m.doc() =
1718
"MLIR All Upstream Extensions, Translations and Passes Registration";
1819

20+
m.def("register_external_models", [](MlirDialectRegistry registry) {
21+
mlirRegisterAllExternalModels(registry);
22+
});
1923
m.def("register_extensions", [](MlirDialectRegistry registry) {
2024
mlirRegisterAllExtensions(registry);
2125
});

mlir/lib/CAPI/Dialect/CMakeLists.txt

Lines changed: 0 additions & 91 deletions
Original file line numberDiff line numberDiff line change
@@ -258,9 +258,6 @@ add_mlir_upstream_c_api_library(MLIRCAPITransformDialect
258258
LINK_LIBS PUBLIC
259259
MLIRCAPIIR
260260
MLIRTransformDialect
261-
MLIRTransformDebugExtension
262-
MLIRTransformLoopExtension
263-
MLIRTransformTuneExtension
264261
)
265262

266263
add_mlir_upstream_c_api_library(MLIRCAPITransformDialectTransforms
@@ -272,94 +269,6 @@ add_mlir_upstream_c_api_library(MLIRCAPITransformDialectTransforms
272269
MLIRTransformDialectTransforms
273270
)
274271

275-
add_mlir_upstream_c_api_library(MLIRCAPIAffineTransformDialectExtension
276-
TransformAffine.cpp
277-
278-
PARTIAL_SOURCES_INTENDED
279-
LINK_LIBS PUBLIC
280-
MLIRAffineTransformOps
281-
)
282-
283-
add_mlir_upstream_c_api_library(MLIRCAPIBufferizationTransformDialectExtension
284-
TransformBufferization.cpp
285-
286-
PARTIAL_SOURCES_INTENDED
287-
LINK_LIBS PUBLIC
288-
MLIRBufferizationTransformOps
289-
)
290-
291-
add_mlir_upstream_c_api_library(MLIRCAPIGPUTransformDialectExtension
292-
TransformGPU.cpp
293-
294-
PARTIAL_SOURCES_INTENDED
295-
LINK_LIBS PUBLIC
296-
MLIRGPUTransformOps
297-
)
298-
299-
add_mlir_upstream_c_api_library(MLIRCAPILinalgTransformDialectExtension
300-
TransformLinalg.cpp
301-
302-
PARTIAL_SOURCES_INTENDED
303-
LINK_LIBS PUBLIC
304-
MLIRLinalgTransformOps
305-
)
306-
307-
add_mlir_upstream_c_api_library(MLIRCAPIMemRefTransformDialectExtension
308-
TransformMemRef.cpp
309-
310-
PARTIAL_SOURCES_INTENDED
311-
LINK_LIBS PUBLIC
312-
MLIRMemRefTransformOps
313-
)
314-
315-
add_mlir_upstream_c_api_library(MLIRCAPINVGPUTransformDialectExtension
316-
TransformNVGPU.cpp
317-
318-
PARTIAL_SOURCES_INTENDED
319-
LINK_LIBS PUBLIC
320-
MLIRNVGPUTransformOps
321-
)
322-
323-
add_mlir_upstream_c_api_library(MLIRCAPIPDLTransformDialectExtension
324-
TransformPDL.cpp
325-
326-
PARTIAL_SOURCES_INTENDED
327-
LINK_LIBS PUBLIC
328-
MLIRTransformPDLExtension
329-
)
330-
331-
add_mlir_upstream_c_api_library(MLIRCAPISMTTransformDialectExtension
332-
TransformSMT.cpp
333-
334-
PARTIAL_SOURCES_INTENDED
335-
LINK_LIBS PUBLIC
336-
MLIRTransformSMTExtension
337-
)
338-
339-
add_mlir_upstream_c_api_library(MLIRCAPISparseTensorTransformDialectExtension
340-
TransformSparseTensor.cpp
341-
342-
PARTIAL_SOURCES_INTENDED
343-
LINK_LIBS PUBLIC
344-
MLIRSparseTensorTransformOps
345-
)
346-
347-
add_mlir_upstream_c_api_library(MLIRCAPITensorTransformDialectExtension
348-
TransformTensor.cpp
349-
350-
PARTIAL_SOURCES_INTENDED
351-
LINK_LIBS PUBLIC
352-
MLIRTensorTransformOps
353-
)
354-
355-
add_mlir_upstream_c_api_library(MLIRCAPIVectorTransformDialectExtension
356-
TransformVector.cpp
357-
358-
PARTIAL_SOURCES_INTENDED
359-
LINK_LIBS PUBLIC
360-
MLIRVectorTransformOps
361-
)
362-
363272
add_mlir_upstream_c_api_library(MLIRCAPIQuant
364273
Quant.cpp
365274

0 commit comments

Comments
 (0)