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

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

As discussed offline, the test cases need to be much simpler



================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3445
+    assert(isInSchedulingRegion(ISD) &&
+           "new ScheduleData already in scheduling region");
+    ScheduleData *SD = allocateScheduleDataChunks();
----------------
Shouldn't the message be "ScheduleData not in scheduling region" ?


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3676
+    doForAllOpcodes(I, [this](ScheduleData *SD) {
+      assert(isInSchedulingRegion(SD));
+      SD->IsScheduled = false;
----------------
assert message: "ScheduleData not in scheduling region" ?


https://reviews.llvm.org/D36518