[PATCH] D36651: [SelectionDAG] Teach the vector-types operand scalarizer about SETCC

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:572
+         N->getOperand(0).getValueType().isVector() &&
+         "Operand types must be vectors");
+
----------------
Assert that the Cond type is v1i1?


================
Comment at: test/CodeGen/X86/pr34177.ll:4
+; RUN: llc < %s -mattr=+avx512f,+avx512vl,+avx512bw,+avx512dq | FileCheck %s --check-prefix=SKX
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
----------------
Is it me or are the two targets' codegen the same? In which case, drop the check-prefix arguments?


https://reviews.llvm.org/D36651