[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 ↗

(On Diff #57651)

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

llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp

 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>();
   assert(BlockToChain.empty());