[PATCH] D109368: [LV] Don't vectorize if we can prove RT + vector cost >= scalar cost.

Wed Sep 8 10:29:15 PDT 2021

lebedev.ri added a comment.

In D109368#2989858 <https://reviews.llvm.org/D109368#2989858>, @ebrevnov wrote:

> Hi Florian,
>
> I do think this change is very important move in the right direction. Thanks for driving it. My main concern is essentially the same as for  D75981 <https://reviews.llvm.org/D75981>. I think such things should be done directly on CostModel. I've uploaded alternative implementation which does the same thing as this patch but in the CostModel. Please take a look and let me know if you like that or not. It would be really helpful to hear from others as well.

(to make it more obvious, the alternative patch is https://reviews.llvm.org/D109444)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109368