This is an archive of the discontinued LLVM Phabricator instance.

[mlir] cleanup transform payload setting
ClosedPublic

Authored by ftynse on Oct 10 2022, 7:41 AM.

Details

Summary

Before the multi-handle relaxation, the transform interpreter did not
actually set payload for a handle that had no further uses as a hacky
mechanism to work around the multi-handle problem. This is no longer
necessary and can be removed to avoid debugging surprises when a handle
has no payload even when its producing op assigned it.

Diff Detail

Event Timeline

ftynse created this revision.Oct 10 2022, 7:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 10 2022, 7:41 AM
ftynse requested review of this revision.Oct 10 2022, 7:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 10 2022, 7:41 AM
This revision is now accepted and ready to land.Oct 10 2022, 11:51 PM
This revision was landed with ongoing or failed builds.Oct 11 2022, 2:55 AM
This revision was automatically updated to reflect the committed changes.