[PATCH] D134982: [X86] Add support for "light" AVX

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D134982#3999687 <https://reviews.llvm.org/D134982#3999687>, @TokarIP wrote:

> In D134982#3999680 <https://reviews.llvm.org/D134982#3999680>, @lebedev.ri wrote:
>
>> I think `LightAVX` is a misnomer. If we want to
>> always utilize full potential of vector load-store unit,
>> then the Tuning should say as much.
>
> I'll probably expand this to other "light" AVX instructions (like vpcmpeq for memcmp intrinsic) in the future.
> Also we don't want the full width, 512-bit load/stores still cause some frequency drop on skylake.

Please don't consider this a blocking feedback, but that sounds
a lot more problematic than just dealing with loads/stores.
But i'll defer to @craig.topper / @RKSimon.


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

https://reviews.llvm.org/D134982