This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Split out Python bindings for dialects into separate libs
ClosedPublic

Authored by ftynse on Jan 5 2022, 7:14 AM.

Details

Summary

Historically, the bindings for the Linalg dialect were included into the
"core" bindings library because they depended on the C++ implementation
of the "core" bindings. The other dialects followed the pattern. Now
that this dependency is gone, split out each dialect into a separate
Python extension library.

Depends On D116649, D116605

Diff Detail

Event Timeline

ftynse created this revision.Jan 5 2022, 7:14 AM
ftynse requested review of this revision.Jan 5 2022, 7:14 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 5 2022, 7:14 AM
stellaraccident accepted this revision.Jan 5 2022, 11:43 AM

Excellent!! Thanks! I think that leaves us only with a hidden dependency on all dialects registration and execution engine that, if fixed, would dramatically reduce downstream build time and distribution size.

This revision is now accepted and ready to land.Jan 5 2022, 11:43 AM
mlir/lib/Bindings/Python/DialectSparseTensor.cpp