[PATCH] D54276: [SelectionDAG][AArch64][X86] Move legalization of vector MULHS/MULHU from LegalizeDAG to LegalizeVectorOps

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 11:21:17 PST 2018


RKSimon added inline comments.


================
Comment at: test/CodeGen/X86/urem-seteq-vec-nonsplat.ll:669
+; CHECK-AVX512VL-NEXT:    vpmuludq %xmm1, %xmm2, %xmm1
+; CHECK-AVX512VL-NEXT:    vpmuludq {{.*}}(%rip), %xmm0, %xmm2
+; CHECK-AVX512VL-NEXT:    vpshufd {{.*#+}} xmm2 = xmm2[1,1,3,3]
----------------
On AVX512 shouldn't these be using the broadcast load fold?


https://reviews.llvm.org/D54276





More information about the llvm-commits mailing list