[PATCH] D54836: [X86] Add DAG combine to combine a v8i32->v8i16 truncate with a packuswb that truncates v8i16->v8i8.

[Craig Topper via Phabricator via llvm-commits]

craig.topper marked an inline comment as done.
craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:35383
+  // create a larger truncate.
+  if (Subtarget.hasAVX512() && Opcode == X86ISD::PACKUS &&
+      N0.getOpcode() == ISD::TRUNCATE && N1.isUndef() && VT == MVT::v16i8 &&
----------------
RKSimon wrote:
> Can the PACKSS equivalent occur as well?
It doesn't occur in any of our lit tests so I'm not sure.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54836/new/

https://reviews.llvm.org/D54836