This is an archive of the discontinued LLVM Phabricator instance.

[mlir][linalg] Add named op for matmul_transpose_b
AcceptedPublic

Authored by ThomasRaoux on Jan 10 2023, 2:00 PM.

Details

Summary

matmul with transposed RHS operand allows better memory access
patterns on several architectures including common GPUs. Having a named
op for it allows to handle this kind of matmul in a more explicit way.

Diff Detail

Event Timeline

ThomasRaoux created this revision.Jan 10 2023, 2:00 PM
ThomasRaoux requested review of this revision.Jan 10 2023, 2:00 PM
hanchung accepted this revision.Jan 10 2023, 2:21 PM
This revision is now accepted and ready to land.Jan 10 2023, 2:21 PM
This revision was automatically updated to reflect the committed changes.
kon72 added a subscriber: kon72.May 6 2023, 12:26 PM

Hello, it appears that the changes made only include the generated yaml file and additional tests. However, it seems that core_named_ops.py was not included. Could you please confirm this?

nicolasvasilache reopened this revision.May 7 2023, 12:20 AM

Yes, this is a problem indeed, the source of truth is the .py file.

This revision is now accepted and ready to land.May 7 2023, 12:20 AM
kon72 added a comment.May 7 2023, 1:46 AM

OK, I can send a fix.

Matt added a subscriber: Matt.Jul 19 2023, 12:53 PM