This is an archive of the discontinued LLVM Phabricator instance.

[MLIR][Linalg] Update filename to reflect implementation (NFC)
ClosedPublic

Authored by chelini on Jul 1 2022, 2:49 AM.

Diff Detail

Event Timeline

chelini created this revision.Jul 1 2022, 2:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 1 2022, 2:49 AM
chelini requested review of this revision.Jul 1 2022, 2:49 AM
nicolasvasilache accepted this revision.Jul 1 2022, 5:41 AM
This revision is now accepted and ready to land.Jul 1 2022, 5:41 AM

The name is confusing now. The pad op is fused with it's producer (not consumer). Could we either revert the change or rename it to FusePadOpWithLinalgProducer

The name is confusing now. The pad op is fused with it's producer (not consumer). Could we either revert the change or rename it to FusePadOpWithLinalgProducer

Updated. I went for FusePadOpWithLinalgProducer.