[PATCH] D28777: [InstCombine][SSE] Add DemandedElts support for PACKSS/PACKUS instructions

[Michael Kuperstein via Phabricator via llvm-commits]

mkuper accepted this revision.
mkuper added a comment.
This revision is now accepted and ready to land.

One more minor naming nit, other than that, LGTM.



================
Comment at: lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:1496
+      // v32i8 PACK(v16i16 X, v16i16 Y) - (X[0..7],Y[0..7]),(X[8..15],Y[8..15])
+      for (int i = 0; i != 2; ++i) {
+        APInt OpDemandedElts(InnerVWidth, 0);
----------------
i -> OpNum?


Repository:
  rL LLVM

https://reviews.llvm.org/D28777