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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 22 06:20:51 PST 2017

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.



More information about the llvm-commits mailing list