[PATCH] D53865: [LoopVectorizer] Improve computation of scalarization overhead.

Tue Nov 27 23:41:47 PST 2018

jonpa added a comment.

In D53865#1310026 <https://reviews.llvm.org/D53865#1310026>, @hsaito wrote:

> Sorry, I must have missed this review.
>
> VPlan based cost modeling (plus VPlan based code motion) should naturally capture this kind of situation ----- but only to the extent that producer/consumer can reside in the same BB. It's taking a lot longer than I wanted to stabilize (compute exactly the same value as existing cost model in LV).

Thanks for taking a look! IIUC, my patch is not useful since VPlan will soon improve this area without it.

I am curious as to how VPlan will accomplish this - will it also add some kind of check with TLI if the instruction will be expanded and propagate this information? Or is there some other way that this may be accomplished?

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

https://reviews.llvm.org/D53865