[PATCH] D52424: [X86] Add AVX512 support to combineVectorSizedSetCCEquality.

[Sanjay Patel via Phabricator via llvm-commits]

spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: test/CodeGen/X86/setcc-wide-types.ll:771-774
 define i32 @ne_i512_pair(i512* %a, i512* %b) {
-; ANY-LABEL: ne_i512_pair:
-; ANY:       # %bb.0:
-; ANY-NEXT:    movq 32(%rdi), %r8
-; ANY-NEXT:    movq 48(%rdi), %r9
-; ANY-NEXT:    movq 40(%rdi), %rdx
-; ANY-NEXT:    movq 56(%rdi), %rcx
-; ANY-NEXT:    xorq 56(%rsi), %rcx
-; ANY-NEXT:    movq 120(%rdi), %rax
-; ANY-NEXT:    xorq 120(%rsi), %rax
-; ANY-NEXT:    orq %rcx, %rax
-; ANY-NEXT:    movq 88(%rdi), %rcx
-; ANY-NEXT:    xorq 88(%rsi), %rcx
-; ANY-NEXT:    orq %rcx, %rax
-; ANY-NEXT:    movq 24(%rdi), %rcx
-; ANY-NEXT:    xorq 24(%rsi), %rcx
-; ANY-NEXT:    xorq 40(%rsi), %rdx
-; ANY-NEXT:    orq %rcx, %rax
-; ANY-NEXT:    movq 104(%rdi), %rcx
-; ANY-NEXT:    xorq 104(%rsi), %rcx
-; ANY-NEXT:    orq %rdx, %rcx
-; ANY-NEXT:    movq 72(%rdi), %rdx
-; ANY-NEXT:    xorq 72(%rsi), %rdx
-; ANY-NEXT:    orq %rdx, %rcx
-; ANY-NEXT:    movq 16(%rdi), %r10
-; ANY-NEXT:    orq %rax, %rcx
-; ANY-NEXT:    movq 8(%rdi), %rax
-; ANY-NEXT:    xorq 8(%rsi), %rax
-; ANY-NEXT:    xorq 48(%rsi), %r9
-; ANY-NEXT:    orq %rax, %rcx
-; ANY-NEXT:    movq 112(%rdi), %rax
-; ANY-NEXT:    xorq 112(%rsi), %rax
-; ANY-NEXT:    orq %r9, %rax
-; ANY-NEXT:    movq 80(%rdi), %rdx
-; ANY-NEXT:    xorq 80(%rsi), %rdx
-; ANY-NEXT:    orq %rdx, %rax
-; ANY-NEXT:    movq (%rdi), %r9
-; ANY-NEXT:    xorq 16(%rsi), %r10
-; ANY-NEXT:    xorq (%rsi), %r9
-; ANY-NEXT:    xorq 32(%rsi), %r8
-; ANY-NEXT:    orq %r10, %rax
-; ANY-NEXT:    movq 96(%rdi), %rdx
-; ANY-NEXT:    movq 64(%rdi), %rdi
-; ANY-NEXT:    xorq 64(%rsi), %rdi
-; ANY-NEXT:    xorq 96(%rsi), %rdx
-; ANY-NEXT:    orq %r8, %rdx
-; ANY-NEXT:    orq %rdi, %rdx
-; ANY-NEXT:    orq %rax, %rdx
-; ANY-NEXT:    orq %r9, %rdx
-; ANY-NEXT:    xorl %eax, %eax
-; ANY-NEXT:    orq %rcx, %rdx
-; ANY-NEXT:    setne %al
-; ANY-NEXT:    retq
+; SSE2-LABEL: ne_i512_pair:
+; SSE2:       # %bb.0:
+; SSE2-NEXT:    movq 32(%rdi), %r8
----------------
Is it worth adding a 'NOT512' prefix to avoid the check duplication?


Repository:
  rL LLVM

https://reviews.llvm.org/D52424