move to function

Author: Meghana Gupta

lib/Backend/FlowGraph.cpp

@@ -4831,14 +4831,7 @@ BasicBlock::CheckLegalityAndFoldPathDepBranches(GlobOpt* globOpt)
             if (currentInlineeEnd != nullptr && currentInlineeEnd != unskippedInlineeEnd)
             {
                 this->GetLastInstr()->InsertBefore(currentInlineeEnd->Copy());
-                if (currentInlineeEnd->m_func->m_hasInlineArgsOpt)
-                {
-                    globOpt->RecordInlineeFrameInfo(currentInlineeEnd);
-                }
-                globOpt->EndTrackingOfArgObjSymsForInlinee();
-
-                Assert(globOpt->currentBlock->globOptData.inlinedArgOutSize >= currentInlineeEnd->GetArgOutSize(/*getInterpreterArgOutCount*/ false));
-                globOpt->currentBlock->globOptData.inlinedArgOutSize -= currentInlineeEnd->GetArgOutSize(/*getInterpreterArgOutCount*/ false);
+                globOpt->ProcessInlineeEnd(currentInlineeEnd);
                 currentInlineeEnd = nullptr;
             }
             // We are adding an unconditional branch, go over all the current successors and remove the ones that are dead now

lib/Backend/GlobOpt.h

@@ -861,6 +861,7 @@ class GlobOpt
     void                    CaptureArguments(BasicBlock *block, BailOutInfo * bailOutInfo, JitArenaAllocator *allocator);
     void                    CaptureByteCodeSymUses(IR::Instr * instr);
     IR::ByteCodeUsesInstr * InsertByteCodeUses(IR::Instr * instr, bool includeDef = false);
+    void                    ProcessInlineeEnd(IR::Instr * instr);
     void                    TrackCalls(IR::Instr * instr);
     void                    RecordInlineeFrameInfo(IR::Instr* instr);
     void                    EndTrackCall(IR::Instr * instr);

lib/Backend/GlobOptBailOut.cpp

@@ -467,6 +467,19 @@ GlobOpt::CaptureByteCodeSymUses(IR::Instr * instr)
         "Instruction edited before capturing the byte code use");
 }
 
+void
+GlobOpt::ProcessInlineeEnd(IR::Instr* instr)
+{
+    if (instr->m_func->m_hasInlineArgsOpt)
+    {
+        RecordInlineeFrameInfo(instr);
+    }
+    EndTrackingOfArgObjSymsForInlinee();
+
+    Assert(this->currentBlock->globOptData.inlinedArgOutSize >= instr->GetArgOutSize(/*getInterpreterArgOutCount*/ false));
+    this->currentBlock->globOptData.inlinedArgOutSize -= instr->GetArgOutSize(/*getInterpreterArgOutCount*/ false);
+}
+
 void
 GlobOpt::TrackCalls(IR::Instr * instr)
 {
@@ -576,14 +589,7 @@ GlobOpt::TrackCalls(IR::Instr * instr)
         break;
 
     case Js::OpCode::InlineeEnd:
-        if (instr->m_func->m_hasInlineArgsOpt)
-        {
-            RecordInlineeFrameInfo(instr);
-        }
-        EndTrackingOfArgObjSymsForInlinee();
-
-        Assert(this->currentBlock->globOptData.inlinedArgOutSize >= instr->GetArgOutSize(/*getInterpreterArgOutCount*/ false));
-        this->currentBlock->globOptData.inlinedArgOutSize -= instr->GetArgOutSize(/*getInterpreterArgOutCount*/ false);
+        ProcessInlineeEnd(instr);
         break;
 
     case Js::OpCode::InlineeMetaArg: