diff options
| author | David Zarzycki <dave@znu.io> | 2019-10-31 12:30:53 +0200 |
|---|---|---|
| committer | David Zarzycki <dave@znu.io> | 2019-11-01 08:58:48 +0200 |
| commit | cb6822c9deb63b6c21263b3732e549fdc89c4bbf (patch) | |
| tree | 13d468e85bb18a1da9b900216f5cb1cab9366be1 /llvm | |
| parent | a8a89c77ea3c16b45763fca6940bbfd3bef7884f (diff) | |
| download | bcm5719-llvm-cb6822c9deb63b6c21263b3732e549fdc89c4bbf.tar.gz bcm5719-llvm-cb6822c9deb63b6c21263b3732e549fdc89c4bbf.zip | |
[X86] Reland: Enable YMM memcmp with AVX1
Update TargetTransformInfo to allow AVX1 to use YMM registers for memcmp.
This is a follow up to D68632 which enabled XOR compares which made this possible.
This also updates the memcmp-optsize.ll test unlike the first patch.
https://reviews.llvm.org/D69658
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/Target/X86/X86TargetTransformInfo.cpp | 5 | ||||
| -rw-r--r-- | llvm/test/CodeGen/X86/memcmp-more-load-pairs.ll | 234 | ||||
| -rw-r--r-- | llvm/test/CodeGen/X86/memcmp-optsize.ll | 51 | ||||
| -rw-r--r-- | llvm/test/CodeGen/X86/memcmp.ll | 96 |
4 files changed, 201 insertions, 185 deletions
diff --git a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp index 2f899b3947b..afbe965ff4f 100644 --- a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp +++ b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp @@ -3444,10 +3444,9 @@ X86TTIImpl::enableMemCmpExpansion(bool OptSize, bool IsZeroCmp) const { // version is not as fast for three way compare (see #33329). const unsigned PreferredWidth = ST->getPreferVectorWidth(); if (PreferredWidth >= 512 && ST->hasAVX512()) Options.LoadSizes.push_back(64); - if (PreferredWidth >= 256 && ST->hasAVX2()) Options.LoadSizes.push_back(32); + if (PreferredWidth >= 256 && ST->hasAVX()) Options.LoadSizes.push_back(32); if (PreferredWidth >= 128 && ST->hasSSE2()) Options.LoadSizes.push_back(16); - // All GPR and vector loads can be unaligned. SIMD compare requires integer - // vectors (SSE2/AVX2). + // All GPR and vector loads can be unaligned. Options.AllowOverlappingLoads = true; } if (ST->is64Bit()) { diff --git a/llvm/test/CodeGen/X86/memcmp-more-load-pairs.ll b/llvm/test/CodeGen/X86/memcmp-more-load-pairs.ll index 139ed023966..ea08e5316ba 100644 --- a/llvm/test/CodeGen/X86/memcmp-more-load-pairs.ll +++ b/llvm/test/CodeGen/X86/memcmp-more-load-pairs.ll @@ -2312,13 +2312,11 @@ define i1 @length32_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length32_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vpxor 16(%rsi), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor (%rsi), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length32_eq: @@ -2653,13 +2651,11 @@ define i1 @length32_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length32_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length32_eq_const: @@ -2814,16 +2810,15 @@ define i1 @length48_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length48_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vmovdqu 32(%rdi), %xmm2 -; X64-AVX1-NEXT: vpxor 16(%rsi), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor (%rsi), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpxor 32(%rsi), %xmm2, %xmm2 -; X64-AVX1-NEXT: vpor %xmm2, %xmm1, %xmm1 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %xmm1 +; X64-AVX1-NEXT: vmovups 32(%rsi), %xmm2 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vxorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length48_eq: @@ -3144,16 +3139,14 @@ define i1 @length48_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length48_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vmovdqu 32(%rdi), %xmm2 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm2, %xmm2 -; X64-AVX1-NEXT: vpor %xmm2, %xmm1, %xmm1 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %xmm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length48_eq_const: @@ -3333,19 +3326,14 @@ define i1 @length63_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length63_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vmovdqu 32(%rdi), %xmm2 -; X64-AVX1-NEXT: vmovdqu 47(%rdi), %xmm3 -; X64-AVX1-NEXT: vpxor 47(%rsi), %xmm3, %xmm3 -; X64-AVX1-NEXT: vpxor 32(%rsi), %xmm2, %xmm2 -; X64-AVX1-NEXT: vpor %xmm3, %xmm2, %xmm2 -; X64-AVX1-NEXT: vpxor 16(%rsi), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpor %xmm2, %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor (%rsi), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 31(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps 31(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length63_eq: @@ -3542,19 +3530,14 @@ define i1 @length63_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length63_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vmovdqu 32(%rdi), %xmm2 -; X64-AVX1-NEXT: vmovdqu 47(%rdi), %xmm3 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm3, %xmm3 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm2, %xmm2 -; X64-AVX1-NEXT: vpor %xmm3, %xmm2, %xmm2 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpor %xmm2, %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 31(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length63_eq_const: @@ -3734,19 +3717,14 @@ define i1 @length64_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length64_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vmovdqu 32(%rdi), %xmm2 -; X64-AVX1-NEXT: vmovdqu 48(%rdi), %xmm3 -; X64-AVX1-NEXT: vpxor 48(%rsi), %xmm3, %xmm3 -; X64-AVX1-NEXT: vpxor 32(%rsi), %xmm2, %xmm2 -; X64-AVX1-NEXT: vpor %xmm3, %xmm2, %xmm2 -; X64-AVX1-NEXT: vpxor 16(%rsi), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpor %xmm2, %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor (%rsi), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps 32(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length64_eq: @@ -3958,19 +3936,14 @@ define i1 @length64_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length64_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vmovdqu 32(%rdi), %xmm2 -; X64-AVX1-NEXT: vmovdqu 48(%rdi), %xmm3 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm3, %xmm3 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm2, %xmm2 -; X64-AVX1-NEXT: vpor %xmm3, %xmm2, %xmm2 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpor %xmm2, %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length64_eq_const: @@ -4073,12 +4046,17 @@ define i1 @length96_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length96_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $96, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vmovups 64(%rdi), %ymm2 +; X64-AVX1-NEXT: vxorps 32(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vxorps 64(%rsi), %ymm2, %ymm2 +; X64-AVX1-NEXT: vorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length96_eq: @@ -4233,13 +4211,17 @@ define i1 @length96_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length96_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $96, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vmovups 64(%rdi), %ymm2 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm2, %ymm2 +; X64-AVX1-NEXT: vorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length96_eq_const: @@ -4355,12 +4337,20 @@ define i1 @length127_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length127_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $127, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vmovups 64(%rdi), %ymm2 +; X64-AVX1-NEXT: vmovups 95(%rdi), %ymm3 +; X64-AVX1-NEXT: vxorps 95(%rsi), %ymm3, %ymm3 +; X64-AVX1-NEXT: vxorps 64(%rsi), %ymm2, %ymm2 +; X64-AVX1-NEXT: vorps %ymm3, %ymm2, %ymm2 +; X64-AVX1-NEXT: vxorps 32(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length127_eq: @@ -4519,13 +4509,20 @@ define i1 @length127_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length127_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $127, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vmovups 64(%rdi), %ymm2 +; X64-AVX1-NEXT: vmovups 95(%rdi), %ymm3 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm3, %ymm3 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm2, %ymm2 +; X64-AVX1-NEXT: vorps %ymm3, %ymm2, %ymm2 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length127_eq_const: @@ -4648,12 +4645,20 @@ define i1 @length128_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length128_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $128, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vmovups 64(%rdi), %ymm2 +; X64-AVX1-NEXT: vmovups 96(%rdi), %ymm3 +; X64-AVX1-NEXT: vxorps 96(%rsi), %ymm3, %ymm3 +; X64-AVX1-NEXT: vxorps 64(%rsi), %ymm2, %ymm2 +; X64-AVX1-NEXT: vorps %ymm3, %ymm2, %ymm2 +; X64-AVX1-NEXT: vxorps 32(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length128_eq: @@ -4812,13 +4817,20 @@ define i1 @length128_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length128_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $128, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vmovups 64(%rdi), %ymm2 +; X64-AVX1-NEXT: vmovups 96(%rdi), %ymm3 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm3, %ymm3 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm2, %ymm2 +; X64-AVX1-NEXT: vorps %ymm3, %ymm2, %ymm2 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length128_eq_const: diff --git a/llvm/test/CodeGen/X86/memcmp-optsize.ll b/llvm/test/CodeGen/X86/memcmp-optsize.ll index 95961bef14f..63bbb8f0ed8 100644 --- a/llvm/test/CodeGen/X86/memcmp-optsize.ll +++ b/llvm/test/CodeGen/X86/memcmp-optsize.ll @@ -684,8 +684,8 @@ define i1 @length24_eq(i8* %x, i8* %y) nounwind optsize { ; X64-AVX-LABEL: length24_eq: ; X64-AVX: # %bb.0: ; X64-AVX-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX-NEXT: vmovq 16(%rdi), %xmm1 -; X64-AVX-NEXT: vmovq 16(%rsi), %xmm2 +; X64-AVX-NEXT: vmovq {{.*#+}} xmm1 = mem[0],zero +; X64-AVX-NEXT: vmovq {{.*#+}} xmm2 = mem[0],zero ; X64-AVX-NEXT: vpxor %xmm2, %xmm1, %xmm1 ; X64-AVX-NEXT: vpxor (%rsi), %xmm0, %xmm0 ; X64-AVX-NEXT: vpor %xmm1, %xmm0, %xmm0 @@ -738,7 +738,7 @@ define i1 @length24_eq_const(i8* %X) nounwind optsize { ; X64-AVX-LABEL: length24_eq_const: ; X64-AVX: # %bb.0: ; X64-AVX-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX-NEXT: vmovq 16(%rdi), %xmm1 +; X64-AVX-NEXT: vmovq {{.*#+}} xmm1 = mem[0],zero ; X64-AVX-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 ; X64-AVX-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 ; X64-AVX-NEXT: vpor %xmm1, %xmm0, %xmm0 @@ -816,13 +816,11 @@ define i1 @length32_eq(i8* %x, i8* %y) nounwind optsize { ; ; X64-AVX1-LABEL: length32_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vpxor 16(%rsi), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor (%rsi), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length32_eq: @@ -878,13 +876,11 @@ define i1 @length32_eq_const(i8* %X) nounwind optsize { ; ; X64-AVX1-LABEL: length32_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length32_eq_const: @@ -944,12 +940,14 @@ define i1 @length64_eq(i8* %x, i8* %y) nounwind optsize { ; ; X64-AVX1-LABEL: length64_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $64, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps 32(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length64_eq: @@ -994,13 +992,14 @@ define i1 @length64_eq_const(i8* %X) nounwind optsize { ; ; X64-AVX1-LABEL: length64_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $64, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length64_eq_const: diff --git a/llvm/test/CodeGen/X86/memcmp.ll b/llvm/test/CodeGen/X86/memcmp.ll index 04f3ade7536..793b4e39284 100644 --- a/llvm/test/CodeGen/X86/memcmp.ll +++ b/llvm/test/CodeGen/X86/memcmp.ll @@ -2007,13 +2007,11 @@ define i1 @length32_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length32_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vpxor 16(%rsi), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor (%rsi), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length32_eq: @@ -2288,13 +2286,11 @@ define i1 @length32_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length32_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: vmovdqu (%rdi), %xmm0 -; X64-AVX1-NEXT: vmovdqu 16(%rdi), %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm1, %xmm1 -; X64-AVX1-NEXT: vpxor {{.*}}(%rip), %xmm0, %xmm0 -; X64-AVX1-NEXT: vpor %xmm1, %xmm0, %xmm0 -; X64-AVX1-NEXT: vptest %xmm0, %xmm0 +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length32_eq_const: @@ -2373,12 +2369,15 @@ define i1 @length48_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length48_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $48, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %xmm1 +; X64-AVX1-NEXT: vmovups 32(%rsi), %xmm2 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vxorps %ymm2, %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length48_eq: @@ -2531,13 +2530,14 @@ define i1 @length48_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length48_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $48, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %xmm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length48_eq_const: @@ -2625,12 +2625,14 @@ define i1 @length63_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length63_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $63, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 31(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps 31(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length63_eq: @@ -2754,13 +2756,14 @@ define i1 @length63_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length63_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $63, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 31(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length63_eq_const: @@ -2848,12 +2851,14 @@ define i1 @length64_eq(i8* %x, i8* %y) nounwind { ; ; X64-AVX1-LABEL: length64_eq: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $64, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps 32(%rsi), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps (%rsi), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: setne %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length64_eq: @@ -2992,13 +2997,14 @@ define i1 @length64_eq_const(i8* %X) nounwind { ; ; X64-AVX1-LABEL: length64_eq_const: ; X64-AVX1: # %bb.0: -; X64-AVX1-NEXT: pushq %rax -; X64-AVX1-NEXT: movl $.L.str, %esi -; X64-AVX1-NEXT: movl $64, %edx -; X64-AVX1-NEXT: callq memcmp -; X64-AVX1-NEXT: testl %eax, %eax +; X64-AVX1-NEXT: vmovups (%rdi), %ymm0 +; X64-AVX1-NEXT: vmovups 32(%rdi), %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm1, %ymm1 +; X64-AVX1-NEXT: vxorps {{.*}}(%rip), %ymm0, %ymm0 +; X64-AVX1-NEXT: vorps %ymm1, %ymm0, %ymm0 +; X64-AVX1-NEXT: vptest %ymm0, %ymm0 ; X64-AVX1-NEXT: sete %al -; X64-AVX1-NEXT: popq %rcx +; X64-AVX1-NEXT: vzeroupper ; X64-AVX1-NEXT: retq ; ; X64-AVX2-LABEL: length64_eq_const: |

