[PATCH] D70456: [Matrix] Add first set of matrix intrinsics and initial lowering pass.

Gerolf Hoflehner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 5 11:11:38 PST 2019


Gerolf added a comment.

LGTM.



================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:45
+// the start address of column \p Col with type (\p EltType x \p NumRows)
+// assuming \p Stride elements between the end and start of two consecutive
+// columns.
----------------
Nit: \p Stride elements. You can delete the rest since you repeat the Stride definition below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70456





More information about the llvm-commits mailing list