[PATCH] D66708: [X86] Pass v32i16/v64i8 in zmm registers on KNL target.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D66708#1649274 <https://reviews.llvm.org/D66708#1649274>, @RKSimon wrote:

> Add a release notes description?
>
> Do we need to flag anything as an ABI change?
>
> Do we have any KNL hardware test coverage?


Where should I flag the ABI change? Hopefully this doesn't have a big impact since using __m512i always work. Its only if you declared your own vector type and passed it to a translation unit compiled with an older compiler that things would break.

Clang was sadly never pushed by Intel for compiling for KNL. Only icc and maybe gcc were. We don't do any testing internally with KNL on clang that I know of.


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

https://reviews.llvm.org/D66708