[PATCH] D81308: [Matrix] Use TileInfo to create tiled loop nest for matrix multiply.

Adam Nemet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 09:54:18 PDT 2020


anemet accepted this revision.
anemet added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: llvm/test/Transforms/LowerMatrixIntrinsics/multiply-fused-volatile.ll:2
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
-; RUN: opt -lower-matrix-intrinsics -fuse-matrix-tile-size=2 -matrix-allow-contract -force-fuse-matrix -verify-dom-info %s -S | FileCheck %s
+; RUN: opt -lower-matrix-intrinsics -fuse-matrix-use-loops -fuse-matrix-tile-size=2 -matrix-allow-contract -force-fuse-matrix -verify-dom-info %s -S | FileCheck %s
 
----------------
Did you also want this -fuse-matrix-use-loops=false ?

I am fine either way, just want to double-check your intentions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81308





More information about the llvm-commits mailing list