[PATCH] D28949: [InstCombine][SSE] Add support for PACKSS/PACKUS constant folding

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:539
+  auto *Cst1 = dyn_cast<Constant>(Arg1);
+  if (Cst0 && Cst1) {
+    SmallVector<Constant *, 32> Vals;
----------------
craig.topper wrote:
> Can you reverse this if as an early out?
Sure - I was hoping to extend this at some point to support non-constant cases where saturation won't happen (and can be safely converted to a shuffle + truncate) but that can wait for now.


Repository:
  rL LLVM

https://reviews.llvm.org/D28949