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

Matthew Simpson via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 11:26:26 PDT 2016


mssimpso added a comment.

LGTM, with a minor comment.


================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1458
@@ -1457,3 +1454,3 @@
 
   if (skipFunction(*F.getFunction()))
     return false;
----------------
My preference would be to remove the second check instead of the first. This would match the other passes.


Repository:
  rL LLVM

http://reviews.llvm.org/D20377





More information about the llvm-commits mailing list