This is an archive of the discontinued LLVM Phabricator instance.

[mlir] NFC - Add a builder to vector.transpose
ClosedPublic

Authored by nicolasvasilache on May 20 2020, 3:32 PM.

Details

Summary

Also expose some more vector ops to EDSCs.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald Transcript
aartbik accepted this revision.May 20 2020, 4:44 PM
aartbik added inline comments.
mlir/include/mlir/Dialect/Vector/VectorOps.td
1390

nit: transp is current name (that may not be a very good one, but let's stay consistent)

mlir/include/mlir/EDSC/Builders.h
361

Nit: here and below

others use Emits and Returns, so let's use the "s" form of verb in this and comments below

This revision is now accepted and ready to land.May 20 2020, 4:44 PM
nicolasvasilache marked an inline comment as done.

Address review.

This revision was automatically updated to reflect the committed changes.