[PATCH] D31526: InstCombine: Use the InstSimplify hook for shufflevector

[David Majnemer via Phabricator via llvm-commits]

majnemer added a comment.

Could you please add a testcase?



================
Comment at: lib/Transforms/InstCombine/InstCombineVectorOps.cpp:1145-1147
+  if (auto *V = SimplifyShuffleVectorInst(LHS, RHS, SVI.getMask(),
+                                          SVI.getType(), DL, &TLI, &DT, &AC))
+    return replaceInstUsesWith(SVI, V);
----------------
I'd move this above the MadeChange declaration.


Repository:
  rL LLVM

https://reviews.llvm.org/D31526