[PATCH] D70901: [Matrix] Update shape propagation to iterate until done.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 09:01:27 PST 2019


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:461
+        WorkList = propagateShapeForward(WorkList);
+        WorkList = propagateShapeBackward(WorkList);
+      }
----------------
LuoYuanke wrote:
> Do we miss the original op ( matrix_multiply, matrix_transpose ...) in backward?
No, we add all instructions we update shape information for in propagateShapeForward, so multiply, transpose & co will be added there as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70901





More information about the llvm-commits mailing list