[PATCH] D130177: [Matrix] Simplify matmuls with scalars

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 09:32:53 PDT 2022


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:104
+template <typename LTy, typename RTy>
+inline auto m_AnyMul(const LTy &L, const RTy &R) {
+  return m_CombineOr(m_Mul(L, R), m_FMul(L, R));
----------------
nit: is `inline` the right thing here or should this be `static` instead?


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

https://reviews.llvm.org/D130177



More information about the llvm-commits mailing list