[PATCH] D35700: DAGCombiner: Extend reduceBuildVecToTrunc to handle non-zero offset

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:35810
+  // TODO: After we improve lowerBuildVector, add execption for VPERMW.
+  return (SrcVT.getScalarSizeInBits() != 32 && Subtarget.hasAVX2());
+
----------------
No, this should be an early-out:
```
if (SrcVT.getScalarSizeInBits() == 32 || !Subtarget.hasAVX2())
  return false;
```


https://reviews.llvm.org/D35700