[PATCH] D49491: [RFC][VPlan, SLP] Add simple SLP analysis on top of VPlan.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 20 03:09:37 PDT 2018


RKSimon added inline comments.


================
Comment at: lib/Transforms/Vectorize/VPlanSLP.cpp:175
+  default: {
+    auto *VPI = cast<VPInstruction>(Values[0]);
+    for (unsigned i = 0, numOps = VPI->getNumOperands(); i < numOps; i++)
----------------
You can move this above the switch statement and remove the duplicate cast


https://reviews.llvm.org/D49491





More information about the llvm-commits mailing list