[PATCH] D26322: [AVX-512] Make VBMI instruction set enabling imply that the BWI instruction set is also enabled.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 23:19:48 PDT 2016


craig.topper created this revision.
craig.topper added reviewers: zvi, delena.
craig.topper added a subscriber: llvm-commits.

This is needed to make the v64i8 and v32i16 types legal for the 512-bit VBMI instructions. Fixes PR30912.

This is the LLVM side part of this. Clang part is here https://reviews.llvm.org/D26306

I removed the -mcpu=skx from the vbmi intrinsic test cases so that we could be sure vbmi was enabling bwi to make the types legal. Somehow that changed the order of the addb instructions. Maybe something to do with the scheduler model?


https://reviews.llvm.org/D26322

Files:
  lib/Target/X86/X86.td
  test/CodeGen/X86/avx512vbmi-intrinsics.ll
  test/CodeGen/X86/avx512vbmivl-intrinsics.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26322.76964.patch
Type: text/x-patch
Size: 12555 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161105/56b9ff74/attachment.bin>


More information about the llvm-commits mailing list