[PATCH] D38683: [X86][AVX512] lowering broadcastm intrinsic - clang part

[jina via Phabricator via cfe-commits]

jina.nahias added inline comments.


================
Comment at: test/CodeGen/avx512cdintrin.c:106
+  // CHECK: insertelement <16 x i32> %{{.*}}, i32 %{{.*}}
+  return _mm512_broadcastmw_epi32(_mm512_cmpeq_epi32_mask ( a, b)); 
 }
----------------
RKSimon wrote:
> Any reason why you can't use the actual insertion indices instead of regexps?
yes, 
the cmpeq intrinsic returns a mask type.
if we wont send a mask type to the broadcastm intrinsic it would do a regular broadcast (not a mask broadcast).


https://reviews.llvm.org/D38683