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

Adam Nemet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 13:30:52 PST 2019


anemet added inline comments.


================
Comment at: llvm/docs/LangRef.rst:14491-14492
+
+The <Rows> and <Cols> arguments must be constant integers. The returned vector
+must have <Rows> * <Cols> elements. %Stride must be >= <Rows>.
+
----------------
We should also be explicit that the unit for %Stride is also the element type.  "Distance" above is a bit ambiguous. 

The rest of the new changes look good.


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