This is an archive of the discontinued LLVM Phabricator instance.

[mlir][linalg] Remove special case for contraction vectorization
ClosedPublic

Authored by ThomasRaoux on Oct 20 2021, 12:38 PM.

Details

Summary

Handle contraction op like all the other generic op reductions. This
simplifies the code. We now rely on contractionOp canonicalization to
keep the same code quality.

Diff Detail

Event Timeline

ThomasRaoux created this revision.Oct 20 2021, 12:38 PM
ThomasRaoux requested review of this revision.Oct 20 2021, 12:38 PM
nicolasvasilache accepted this revision.Oct 20 2021, 1:05 PM

it's very nice that this becomes as simple as a red diff once the missing canonicalizations work as intended!

This revision is now accepted and ready to land.Oct 20 2021, 1:05 PM
This revision was landed with ongoing or failed builds.Oct 21 2021, 2:11 PM
This revision was automatically updated to reflect the committed changes.