[PATCH] D102748: [LoopUnroll] Don't unroll before vectorisation

Wed Jun 2 04:47:10 PDT 2021

xbolva00 added a comment.

In D102748#2791077 <https://reviews.llvm.org/D102748#2791077>, @xbolva00 wrote:

> In D102748#2778813 <https://reviews.llvm.org/D102748#2778813>, @xbolva00 wrote:
>
>> And there is something weird in terms of cost modelling between vectorizers.
>>
>> See https://bugs.llvm.org/show_bug.cgi?id=47491
>>
>> If we dont unroll the loop, llvm loop vectorizer vectorizes the loop (cost model says it is profitable).
>>
>> If we unroll the loop, slp does not vectorize the unrolled loop.
>
> And other one: https://bugs.llvm.org/show_bug.cgi?id=50552 ... SLP just misses currently these patterns.

But yeah, can be fixed with @fhahn's D102834 <https://reviews.llvm.org/D102834>

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

https://reviews.llvm.org/D102748