This is an archive of the discontinued LLVM Phabricator instance.

[mlir][linalg][transform][python] Simplify mix-in of PadOp.
ClosedPublic

Authored by ingomueller-net on Sep 4 2023, 1:39 AM.

Details

Summary

This patch removes some manual conversion of mixed Python/attribute
arguments to I64ArrayAttrs, which turned out to be unnecessary.
Interestingly, this change does not depend on the additional attribute
builders added in the (currently pending)
https://reviews.llvm.org/D159403 patch.

Diff Detail

Event Timeline

ingomueller-net created this revision.Sep 4 2023, 1:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 4 2023, 1:39 AM
ingomueller-net requested review of this revision.Sep 4 2023, 1:39 AM
ftynse accepted this revision.Sep 4 2023, 1:51 AM

Consider using Github for future patches, we are moving away from Phabricator.

This revision is now accepted and ready to land.Sep 4 2023, 1:51 AM

Oh, yeah, thanks for the reminder! Will do!