[PATCH] D41436: [X86][AVX512] Enable variable shuffle combining by default on AVX512 targets

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a reviewer: hfinkel.
RKSimon added a subscriber: hfinkel.
RKSimon added a comment.

In https://reviews.llvm.org/D41436#969513, @RKSimon wrote:

> In https://reviews.llvm.org/D41436#969481, @zvi wrote:
>
> > Still trying to get hold of a KNL expert that will answer whether KNL should be included. Can we for now conservatively assume no and exclude KNL from this patch just so this patch can make progress? I want to follow-up on updating the AVX2 tests with FastVariableShuffle configurations.
>
>
>


@hfinkel IIRC you've been using KNL, should we be trying to combine to a variable shuffle to combine 2 shuffles or more?


Repository:
  rL LLVM

https://reviews.llvm.org/D41436