[PATCH] D43932: [Polly][GEMM] Include parameter constraints in isMatMulOperandAcc

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 3 02:00:33 PST 2018


Meinersbur added a comment.

LGTM, but I cannot apply the patch. I assume you uploaded a diff relative to your own version. E.g. there is no newline in at line 1159 in trunk.

It would be nice to have a commit message describing why this is changes, what the intended effect is, and how the change facilitates the effect.


https://reviews.llvm.org/D43932





More information about the llvm-commits mailing list