[llvm] 443c5b9 - [NFC] Remove redundant check for MBB being empty in outliner

Jessica Paquette via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 16:41:56 PST 2023


Author: Jessica Paquette
Date: 2023-02-03T16:41:02-08:00
New Revision: 443c5b9fd505f5fe2df92ee27bf1d4d9e0dc4cb9

URL: https://github.com/llvm/llvm-project/commit/443c5b9fd505f5fe2df92ee27bf1d4d9e0dc4cb9
DIFF: https://github.com/llvm/llvm-project/commit/443c5b9fd505f5fe2df92ee27bf1d4d9e0dc4cb9.diff

LOG: [NFC] Remove redundant check for MBB being empty in outliner

If the size is < 2, then we just break anyway.

Added: 
    

Modified: 
    llvm/lib/CodeGen/MachineOutliner.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/MachineOutliner.cpp b/llvm/lib/CodeGen/MachineOutliner.cpp
index aac60d616f05..7282a4c13c2f 100644
--- a/llvm/lib/CodeGen/MachineOutliner.cpp
+++ b/llvm/lib/CodeGen/MachineOutliner.cpp
@@ -956,7 +956,7 @@ void MachineOutliner::populateMapper(InstructionMapper &Mapper, Module &M,
       // contain something worth outlining.
       // FIXME: This should be based off of the maximum size in B of an outlined
       // call versus the size in B of the MBB.
-      if (MBB.empty() || MBB.size() < 2)
+      if (MBB.size() < 2)
         continue;
 
       // Check if MBB could be the target of an indirect branch. If it is, then


        


More information about the llvm-commits mailing list