[PATCH] D72734: [VectorUtils] Rework the Vector Function Database (VFDatabase).

Wed Jan 15 09:35:07 PST 2020

fpetrogalli added a comment.

In D72734#1821252 <https://reviews.llvm.org/D72734#1821252>, @uabelho wrote:

> In D72734#1820557 <https://reviews.llvm.org/D72734#1820557>, @fpetrogalli wrote:
>
> > @uabelho , let me know if the introduction of `isTLIScalarize` prevents the regressions you have seen in your out-of-tree compiler.
>
>
> Hi!
>
> I applied this new patch as is and now I don't see the regressions I saw last time.

Great! Happy to hear that. The patch doesn't differ from the original one at https://reviews.llvm.org/D67572, other than for the addition of the `isTLIScalarize` method and its use in `LoopVectorizationLegality.cpp`. If you are happy with it, may I ask you to formally approve it so I can submit it? (Feel free to do a full review if you prefer to do so!)

Kind regards,

Francesco

> Nice!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72734