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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 05:19:44 PDT 2017

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" ?


More information about the llvm-commits mailing list