[PATCH] D102733: [Matrix] Factor and distribute transposes across multiplies

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 25 08:35:21 PDT 2021


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:207
     unsigned NumComputeOps = 0;
+    // Most of the time transposes can be fused with matrix multiplies or can be
+    // folded away via algebraic simplifications.  This is the number of
----------------
nit: `///` for doc-comment as the other members?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102733/new/

https://reviews.llvm.org/D102733



More information about the llvm-commits mailing list