[PATCH] D25874: [X86][SSE] Add support for (V)PMOVSX* constant folding

[Zvi Rackover via llvm-commits]

zvi added a comment.

LGTM



================
Comment at: lib/Target/X86/X86ISelLowering.cpp:31809
+    SmallVector<APInt, 4> Vals(NumDstElts, APInt(SVT.getSizeInBits(), 0));
+    for (int i = 0; i != NumDstElts; ++i) {
       SDValue OpElt = Op.getOperand(i);
----------------
'i' and 'NumDstElts' are of different types, so the comparison fire a warning.


Repository:
  rL LLVM

https://reviews.llvm.org/D25874