maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Revert "[mlir] fix crash in PybindAdaptors.h" 

This reverts commit 970cb57ef7.

Broke the buildbot.
This commit is contained in:
Alex Zinenko 2022-01-17 10:55:36 +01:00
parent 873a7ee7e4
commit 2325f36301
1 changed files with 4 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
mlir/include/mlir/Bindings/Python/PybindAdaptors.h
View File

@ -320,11 +320,7 @@ public:
// Casting constructor. Note that defining an __init__ method is special
// and not yet generalized on pure_subclass (it requires a somewhat
// different cpp_function and other requirements on chaining to super
// __init__ make it more awkward to do generally). It is marked as
// `is_new_style_constructor` to suppress the deprecation warning from
// pybind11 related to placement-new since we are not doing any allocation
// here but relying on the superclass constructor that does "new-style"
// allocation for pybind11.
// __init__ make it more awkward to do generally).
std::string captureTypeName(
typeClassName); // As string in case if typeClassName is not static.
py::cpp_function initCf(
@ -340,9 +336,7 @@ public:
}
superClass.attr("__init__")(self, otherType);
},
py::name("__init__"), py::arg("cast_from_type"),
py::is_method(scope.attr(typeClassName)),
py::detail::is_new_style_constructor(),
py::arg("cast_from_type"), py::is_method(py::none()),
"Casts the passed type to this specific sub-type.");
thisClass.attr("__init__") = initCf;
@ -377,11 +371,7 @@ public:
// Casting constructor. Note that defining an __init__ method is special
// and not yet generalized on pure_subclass (it requires a somewhat
// different cpp_function and other requirements on chaining to super
// __init__ make it more awkward to do generally). It is marked as
// `is_new_style_constructor` to suppress the deprecation warning from
// pybind11 related to placement-new since we are not doing any allocation
// here but relying on the superclass constructor that does "new-style"
// allocation for pybind11.
// __init__ make it more awkward to do generally).
std::string captureTypeName(
typeClassName); // As string in case if typeClassName is not static.
py::cpp_function initCf(
@ -397,9 +387,7 @@ public:
}
superClass.attr("__init__")(self, otherType);
},
py::name("__init__"), py::arg("cast_from_type"),
py::is_method(scope.attr(typeClassName)),
py::detail::is_new_style_constructor(),
py::arg("cast_from_type"), py::is_method(py::none()),
"Casts the passed type to this specific sub-type.");
thisClass.attr("__init__") = initCf;