[llvm] [SLP][NFC] Redesign schedule bundle, separate from schedule data, NFC (PR #131625)
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 19 06:28:09 PDT 2025
================
@@ -18581,10 +18784,13 @@ void BoUpSLP::scheduleBlock(BlockScheduling *BS) {
#if !defined(NDEBUG) || defined(EXPENSIVE_CHECKS)
// Check that all schedulable entities got scheduled
- for (auto *I = BS->ScheduleStart; I != BS->ScheduleEnd; I = I->getNextNode()) {
- ScheduleData *SD = BS->getScheduleData(I);
- if (SD && SD->isSchedulingEntity() && SD->hasValidDependencies())
- assert(SD->IsScheduled && "must be scheduled at this point");
+ for (auto *I = BS->ScheduleStart; I != BS->ScheduleEnd;
+ I = I->getNextNode()) {
+ ArrayRef<ScheduleBundle *> Bundles = BS->getScheduleBundles(I);
+ if (!Bundles.empty()) {
+ for (ScheduleBundle *Bundle : Bundles)
+ assert(Bundle->isScheduled() && "must be scheduled at this point");
----------------
RKSimon wrote:
Use assert(all_of()) ?
https://github.com/llvm/llvm-project/pull/131625
More information about the llvm-commits
mailing list