[MBP] Remove a redundant skipFunction(). NFC.

[haicheng]

Summary

skipFunction() is called twice in runOnMachineFunction(). Remove one of them.

[mssimpso]

LGTM, with a minor comment.

lib/CodeGen/MachineBlockPlacement.cpp

1458

My preference would be to remove the second check instead of the first. This would match the other passes.

[haicheng]

Address Matt's comment. Thank you, Matt.

[andrew.w.kaylor]

lgtm

lib/CodeGen/MachineBlockPlacement.cpp

BranchProbability LayoutProb =
         MBPI->getEdgeProbability(LayoutPred, ChainBB);
     BlockFrequency LayoutEdgeFreq = MBFI->getBlockFreq(LayoutPred) * LayoutProb;
     if (LayoutEdgeFreq <= (Freq * ColdProb))
       ChainBB->setAlignment(Align);
   }
 }
 
 bool MachineBlockPlacement::runOnMachineFunction(MachineFunction &F) {
-  if (skipFunction(*F.getFunction()))
-    return false;
-
   // Check for single-block functions and skip them.
   if (std::next(F.begin()) == F.end())
     return false;
 
   if (skipFunction(*F.getFunction()))
     return false;
 
   MBPI = &getAnalysis<MachineBranchProbabilityInfo>();
   MBFI = &getAnalysis<MachineBlockFrequencyInfo>();
   MLI = &getAnalysis<MachineLoopInfo>();
   TII = F.getSubtarget().getInstrInfo();
   TLI = F.getSubtarget().getTargetLowering();
   MDT = &getAnalysis<MachineDominatorTree>();