[PATCH] D20377: [MBP] Remove a redundant skipFunction(). NFC.

Haicheng Wu via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 11:22:15 PDT 2016


haicheng created this revision.
haicheng added reviewers: mcrosier, gberry, mssimpso.
haicheng added subscribers: deadalnix, llvm-commits, andrew.w.kaylor.
haicheng set the repository for this revision to rL LLVM.
Herald added a subscriber: mcrosier.

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

Repository:
  rL LLVM

http://reviews.llvm.org/D20377

Files:
  lib/CodeGen/MachineBlockPlacement.cpp

Index: lib/CodeGen/MachineBlockPlacement.cpp
===================================================================
--- lib/CodeGen/MachineBlockPlacement.cpp
+++ lib/CodeGen/MachineBlockPlacement.cpp
@@ -1448,9 +1448,6 @@
 }
 
 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;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20377.57651.patch
Type: text/x-patch
Size: 469 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160518/2ebf56a8/attachment.bin>


More information about the llvm-commits mailing list