[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