diff options
| -rw-r--r-- | lib/Transforms/IPO/InlineSimple.cpp | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/lib/Transforms/IPO/InlineSimple.cpp b/lib/Transforms/IPO/InlineSimple.cpp index d88a923e22..41f849a798 100644 --- a/lib/Transforms/IPO/InlineSimple.cpp +++ b/lib/Transforms/IPO/InlineSimple.cpp @@ -190,7 +190,18 @@ bool InlineFunction(CallInst *CI) { RemapInstruction(II, ValueMap); } - if (PHI) RemapInstruction(PHI, ValueMap); // Fix the PHI node also... + if (PHI) { + RemapInstruction(PHI, ValueMap); // Fix the PHI node also... + + // Check to see if the PHI node only has one argument. This is a common + // case resulting from there only being a single return instruction in the + // function call. Because this is so common, eliminate the PHI node. + // + if (PHI->getNumIncomingValues() == 1) { + PHI->replaceAllUsesWith(PHI->getIncomingValue(0)); + PHI->getParent()->getInstList().erase(PHI); + } + } // Change the branch that used to go to NewBB to branch to the first basic // block of the inlined function. |