[PATCH] D12317: AVX512 : vpextrb/w/d/q and vpinsrb/w/d/q implementation

[Elena Demikhovsky via llvm-commits]

delena added a comment.

LGTM


================
Comment at: lib/Target/X86/X86InstrSSE.td:4614
@@ -4613,3 +4613,3 @@
 // Extract
-let Predicates = [HasAVX] in
+let Predicates = [HasAVX, NoBWI] in
 def VPEXTRWri : Ii8<0xC5, MRMSrcReg,
----------------
Please add tests for AVX


Repository:
  rL LLVM

http://reviews.llvm.org/D12317