[PATCH] D67081: [ModuloSchedule] Introduce PeelingModuloScheduleExpander

Thomas Raoux via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 16:50:53 PDT 2019


ThomasRaoux accepted this revision.
ThomasRaoux added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/ModuloSchedule.cpp:1400
+    // rescheduled loop.
+    assert(LoopProducerCycle <= ConsumerCycle);
+    assert(LoopProducerStage == ConsumerStage + 1);
----------------
Thanks for explaining. Those properties are enforced by the swing scheduler ASAP and ALAP functions? Maybe a comment would make it easier to understand that part? 


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67081/new/

https://reviews.llvm.org/D67081





More information about the llvm-commits mailing list