[PATCH] D57382: [LV] Move interleave count computation to LVP::plan().

Wed Feb 6 12:33:17 PST 2019

fhahn added a comment.

In D57382#1387174 <https://reviews.llvm.org/D57382#1387174>, @Ayal wrote:

> In D57382#1387166 <https://reviews.llvm.org/D57382#1387166>, @fhahn wrote:
>
> > Thanks Ayal! Do I understand correctly that you propose to first fix the missing VPlan issue by make interleaving respect the fact that interleaving should be avoided when computeMaxVF returns None? If that is the intended behavior, then I think it makes sense to fix that first and I'll prepare a separate patch.
>
>
> Yes, that's right. Just curious if that'll indeed fix the original bug.

Yep it fixes the original bug. I've created D57837 <https://reviews.llvm.org/D57837>. Once that is resolved, I'll update this patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57382