[PATCH] D36518: [SLPVectorizer] Schedule bundle with different opcodes.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:984
+              // decrementing,
+              // so we can put the dependent instruction into the ready list.
+              ScheduleData *DepBundle = OpDef->FirstInBundle;
----------------
tidyup the comment


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3713
+             "scheduler and vectorizer have different opinion on what is a "
+             "bundle");
+      SD->FirstInBundle->SchedulingPriority = Idx++;
----------------
possibly rephrase the assertion message: "scheduler and vectorizer bundle mismatch"?


https://reviews.llvm.org/D36518