[PATCH] D75566: [Matrix] Add initial tiling for load/multiply/store chains.

[Adam Nemet via Phabricator via llvm-commits]

anemet added a comment.

Very nice!



================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:51-53
+static cl::opt<bool> FuseMatrix("fuse-matrix", cl::init(true));
+static cl::opt<unsigned> TileSize("fuse-matrix-tile-size", cl::init(4));
+static cl::opt<bool> ForceFusion("force-fuse-matrix", cl::init(false));
----------------
Add cl::desc


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:970
+
+    Value *St_b =
+        Builder.CreatePtrToInt(const_cast<Value *>(St.Ptr), IntPtrTy, "st_b");
----------------
Please add a comment explaining the code that is generated below this point



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75566