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

Zvi Rackover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 31 01:47:29 PDT 2017


zvi added inline comments.


================
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);
----------------
majnemer wrote:
> I'd move this above the MadeChange declaration.
Sure


Repository:
  rL LLVM

https://reviews.llvm.org/D31526





More information about the llvm-commits mailing list