address comments

Author: c-rhodes

llvm/lib/CodeGen/MachineScheduler.cpp

@@ -3512,7 +3512,7 @@ void GenericScheduler::initPolicy(MachineBasicBlock::iterator Begin,
     RegionPolicy.OnlyTopDown = false;
   }
 
-  this->BotIdx = NumRegionInstrs - 1;
+  BotIdx = NumRegionInstrs - 1;
   this->NumRegionInstrs = NumRegionInstrs;
 }
 
@@ -3995,6 +3995,7 @@ SUnit *GenericScheduler::pickNode(bool &IsTopNode) {
     if (SU->NodeNum == TopIdx++)
       ++NumInstrsInSourceOrderPreRA;
   } else {
+    assert(BotIdx < NumRegionInstrs && "out of bounds");
     if (SU->NodeNum == BotIdx--)
       ++NumInstrsInSourceOrderPreRA;
   }
@@ -4348,6 +4349,7 @@ SUnit *PostGenericScheduler::pickNode(bool &IsTopNode) {
     if (SU->NodeNum == TopIdx++)
       ++NumInstrsInSourceOrderPostRA;
   } else {
+    assert(BotIdx < NumRegionInstrs && "out of bounds");
     if (SU->NodeNum == BotIdx--)
       ++NumInstrsInSourceOrderPostRA;
   }