[PATCH] D102733: [Matrix] Factor and distribute transposes across multiplies

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 20 06:32:16 PDT 2021


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:688
+        // If we were to erase II, move again.
+        auto eraseFromParent = [&](Value *V) {
+          auto *Inst = cast<Instruction>(V);
----------------
It should be enough to capture `&II`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102733



More information about the llvm-commits mailing list