This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Remove Linalg fusion-on-memrefs.
ClosedPublic

Authored by pifon2a on Jan 15 2023, 3:31 PM.

Diff Detail

Event Timeline

pifon2a created this revision.Jan 15 2023, 3:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 15 2023, 3:31 PM
pifon2a requested review of this revision.Jan 15 2023, 3:31 PM
mravishankar accepted this revision.Jan 15 2023, 6:55 PM

Nice cleanup!

This revision is now accepted and ready to land.Jan 15 2023, 6:55 PM
mravishankar requested changes to this revision.Jan 15 2023, 6:57 PM

Actually when I deleted some similar code I posted a PSA for it (https://discourse.llvm.org/t/psa-retire-tileandfuselinalgops-method/63850). There might be users of this that might be using this in a load-bearing fashion. Better to get community input first.

This revision now requires changes to proceed.Jan 15 2023, 6:57 PM

Actually when I deleted some similar code I posted a PSA for it (https://discourse.llvm.org/t/psa-retire-tileandfuselinalgops-method/63850). There might be users of this that might be using this in a load-bearing fashion. Better to get community input first.

Sure, let's have a PSA first. https://discourse.llvm.org/t/psa-retire-linalg-fusion-on-memrefs/67720

mravishankar accepted this revision.Jan 20 2023, 9:42 AM

unblocking since there was a PSA. Thanks!

This revision is now accepted and ready to land.Jan 20 2023, 9:42 AM
This revision was automatically updated to reflect the committed changes.
mlir/test/Dialect/Linalg/fusion-2-level.mlir