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

Fri Jul 17 13:36:41 PDT 2020

anemet accepted this revision.
anemet added a comment.
This revision is now accepted and ready to land.

In D76867#2156196 <https://reviews.llvm.org/D76867#2156196>, @fhahn wrote:

> 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?

Maybe that that is in a separate patch, up to you.

LGTM!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76867