[PATCH] D76867: [Matrix] Add minimal lowering pass that only requires TTI.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 16 09:41:47 PDT 2020


fhahn updated this revision to Diff 278523.
fhahn added a comment.

In D76867#2154836 <https://reviews.llvm.org/D76867#2154836>, @anemet wrote:

> Is this used anywhere in the patch?  If not please explain the plan in the description.


I've updated the O0 case to use it (just committed that today). D76858 <https://reviews.llvm.org/D76858> will use it in the backend pipeline. I think the description should already mention that the intention is to use it in the backend pipeline, without having to compute DT/AA there. Should I add more details?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76867

Files:
  llvm/include/llvm/InitializePasses.h
  llvm/include/llvm/Transforms/Scalar.h
  llvm/lib/Transforms/IPO/PassManagerBuilder.cpp
  llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp
  llvm/lib/Transforms/Scalar/Scalar.cpp
  llvm/test/Transforms/LowerMatrixIntrinsics/multiply-minimal.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76867.278523.patch
Type: text/x-patch
Size: 14497 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200716/61fac40d/attachment-0001.bin>


More information about the llvm-commits mailing list