[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