Change usedCapturedValues in BailOutInfo to a pointer for consistency

nhat-nguyen · nhat-nguyen · commit 6abc0581bdb4 · 2018-10-15T14:35:53.000-07:00
As part of `LazyBailOut`, we want to perform a *shallow* copy of a `BailOutInfo` which includes sharing all field pointers. However, `usedCapturedValues` is the only field in `BailOutInfo` that is not a pointer and is initialized together with `CapturedValues`, which makes it more complicated to do so as we have to perform manual copy of each field inside `CapturedValues`. With this change, we are more consistent with what we already have and also make it easier for future `LazyBailOut` work.
diff --git a/lib/Backend/BackwardPass.cpp b/lib/Backend/BackwardPass.cpp
@@ -1763,19 +1763,19 @@ BackwardPass::ProcessBailOutArgObj(BailOutInfo * bailOutInfo, BVSparse<JitArenaA
         {
             if (byteCodeUpwardExposedUsed->TestAndClear(symId))
             {
-                if (bailOutInfo->usedCapturedValues.argObjSyms == nullptr)
+                if (bailOutInfo->usedCapturedValues->argObjSyms == nullptr)
                 {
-                    bailOutInfo->usedCapturedValues.argObjSyms = JitAnew(this->func->m_alloc,
+                    bailOutInfo->usedCapturedValues->argObjSyms = JitAnew(this->func->m_alloc,
                         BVSparse<JitArenaAllocator>, this->func->m_alloc);
                 }
-                bailOutInfo->usedCapturedValues.argObjSyms->Set(symId);
+                bailOutInfo->usedCapturedValues->argObjSyms->Set(symId);
             }
         }
         NEXT_BITSET_IN_SPARSEBV;
     }
-    if (bailOutInfo->usedCapturedValues.argObjSyms)
+    if (bailOutInfo->usedCapturedValues->argObjSyms)
     {
-        byteCodeUpwardExposedUsed->Minus(bailOutInfo->usedCapturedValues.argObjSyms);
+        byteCodeUpwardExposedUsed->Minus(bailOutInfo->usedCapturedValues->argObjSyms);
     }
 }
 
@@ -1785,7 +1785,7 @@ BackwardPass::ProcessBailOutConstants(BailOutInfo * bailOutInfo, BVSparse<JitAre
     Assert(this->tag != Js::BackwardPhase);
 
     // Remove constants that we are already going to restore
-    SListBase<ConstantStackSymValue> * usedConstantValues = &bailOutInfo->usedCapturedValues.constantValues;
+    SListBase<ConstantStackSymValue> * usedConstantValues = &bailOutInfo->usedCapturedValues->constantValues;
     FOREACH_SLISTBASE_ENTRY(ConstantStackSymValue, value, usedConstantValues)
     {
         byteCodeUpwardExposedUsed->Clear(value.Key()->m_id);
@@ -1817,7 +1817,7 @@ BackwardPass::ProcessBailOutCopyProps(BailOutInfo * bailOutInfo, BVSparse<JitAre
     Assert(!this->func->GetJITFunctionBody()->IsAsmJsMode());
 
     // Remove copy prop that we were already going to restore
-    SListBase<CopyPropSyms> * usedCopyPropSyms = &bailOutInfo->usedCapturedValues.copyPropSyms;
+    SListBase<CopyPropSyms> * usedCopyPropSyms = &bailOutInfo->usedCapturedValues->copyPropSyms;
     FOREACH_SLISTBASE_ENTRY(CopyPropSyms, copyPropSyms, usedCopyPropSyms)
     {
         byteCodeUpwardExposedUsed->Clear(copyPropSyms.Key()->m_id);
@@ -2596,7 +2596,7 @@ BackwardPass::ProcessBailOutInfo(IR::Instr * instr, BailOutInfo * bailOutInfo)
             tempBv->And(this->func->m_nonTempLocalVars, bailOutInfo->liveVarSyms);
 
             // Remove syms that are restored in other ways than byteCodeUpwardExposedUsed.
-            FOREACH_SLIST_ENTRY(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues.constantValues)
+            FOREACH_SLIST_ENTRY(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues->constantValues)
             {
                 Assert(value.Key()->HasByteCodeRegSlot() || value.Key()->GetInstrDef()->m_opcode == Js::OpCode::BytecodeArgOutCapture);
                 if (value.Key()->HasByteCodeRegSlot())
@@ -2605,7 +2605,7 @@ BackwardPass::ProcessBailOutInfo(IR::Instr * instr, BailOutInfo * bailOutInfo)
                 }
             }
             NEXT_SLIST_ENTRY;
-            FOREACH_SLIST_ENTRY(CopyPropSyms, value, &bailOutInfo->usedCapturedValues.copyPropSyms)
+            FOREACH_SLIST_ENTRY(CopyPropSyms, value, &bailOutInfo->usedCapturedValues->copyPropSyms)
             {
                 Assert(value.Key()->HasByteCodeRegSlot() || value.Key()->GetInstrDef()->m_opcode == Js::OpCode::BytecodeArgOutCapture);
                 if (value.Key()->HasByteCodeRegSlot())
@@ -2614,9 +2614,9 @@ BackwardPass::ProcessBailOutInfo(IR::Instr * instr, BailOutInfo * bailOutInfo)
                 }
             }
             NEXT_SLIST_ENTRY;
-            if (bailOutInfo->usedCapturedValues.argObjSyms)
+            if (bailOutInfo->usedCapturedValues->argObjSyms)
             {
-                tempBv->Minus(bailOutInfo->usedCapturedValues.argObjSyms);
+                tempBv->Minus(bailOutInfo->usedCapturedValues->argObjSyms);
             }
 
             byteCodeUpwardExposedUsed->Or(tempBv);
@@ -8226,7 +8226,7 @@ BackwardPass::ReverseCopyProp(IR::Instr *instr)
             FOREACH_SLISTBASE_ENTRY(
                 CopyPropSyms,
                 usedCopyPropSym,
-                &instrPrev->GetBailOutInfo()->usedCapturedValues.copyPropSyms)
+                &instrPrev->GetBailOutInfo()->usedCapturedValues->copyPropSyms)
             {
                 if(dstSym == usedCopyPropSym.Value())
                 {
diff --git a/lib/Backend/BailOut.cpp b/lib/Backend/BailOut.cpp
@@ -24,8 +24,21 @@ BailOutInfo::Clear(JitArenaAllocator * allocator)
         this->capturedValues->copyPropSyms.Clear(allocator);
         JitAdelete(allocator, this->capturedValues);
     }
-    this->usedCapturedValues.constantValues.Clear(allocator);
-    this->usedCapturedValues.copyPropSyms.Clear(allocator);
+
+    if (this->usedCapturedValues)
+    {
+        Assert(this->usedCapturedValues->refCount == 0);
+        this->usedCapturedValues->constantValues.Clear(allocator);
+        this->usedCapturedValues->copyPropSyms.Clear(allocator);
+
+        if (this->usedCapturedValues->argObjSyms)
+        {
+            JitAdelete(allocator, this->usedCapturedValues->argObjSyms);
+        }
+
+        JitAdelete(allocator, this->usedCapturedValues);
+    }
+
     if (byteCodeUpwardExposedUsed)
     {
         JitAdelete(allocator, byteCodeUpwardExposedUsed);
diff --git a/lib/Backend/BailOut.h b/lib/Backend/BailOut.h
@@ -41,7 +41,9 @@ class BailOutInfo
 #endif
         this->capturedValues = JitAnew(bailOutFunc->m_alloc, CapturedValues);
         this->capturedValues->refCount = 1;
-        this->usedCapturedValues.argObjSyms = nullptr;
+
+        this->usedCapturedValues = JitAnew(bailOutFunc->m_alloc, CapturedValues);
+        this->usedCapturedValues->argObjSyms = nullptr;
     }
     void Clear(JitArenaAllocator * allocator);
 
@@ -77,9 +79,9 @@ class BailOutInfo
 #endif
     uint32 bailOutOffset;
     BailOutRecord * bailOutRecord;
-    CapturedValues* capturedValues;                                      // Values we know about after forward pass
-    CapturedValues usedCapturedValues;                                  // Values that need to be restored in the bail out
-    BVSparse<JitArenaAllocator> * byteCodeUpwardExposedUsed;            // Non-constant stack syms that needs to be restored in the bail out
+    CapturedValues * capturedValues;                                      // Values we know about after forward pass
+    CapturedValues * usedCapturedValues;                                  // Values that need to be restored in the bail out
+    BVSparse<JitArenaAllocator> * byteCodeUpwardExposedUsed;              // Non-constant stack syms that needs to be restored in the bail out
     uint polymorphicCacheIndex;
     uint startCallCount;
     uint totalOutParamCount;
diff --git a/lib/Backend/LinearScan.cpp b/lib/Backend/LinearScan.cpp
@@ -1417,7 +1417,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)
     memset(state.registerSaveSyms, 0, sizeof(state.registerSaveSyms));
 
     // Fill in the constants
-    FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues.constantValues, constantValuesIterator)
+    FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues->constantValues, constantValuesIterator)
     {
         AssertMsg(bailOutInfo->bailOutRecord->bailOutKind != IR::BailOutForGeneratorYield, "constant prop syms unexpected for bail-in for generator yield");
         StackSym * stackSym = value.Key();
@@ -1460,7 +1460,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)
     NEXT_SLISTBASE_ENTRY_EDITING;
 
     // Fill in the copy prop syms
-    FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues.copyPropSyms, copyPropSymsIter)
+    FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues->copyPropSyms, copyPropSymsIter)
     {
         AssertMsg(bailOutInfo->bailOutRecord->bailOutKind != IR::BailOutForGeneratorYield, "copy prop syms unexpected for bail-in for generator yield");
         StackSym * stackSym = copyPropSyms.Key();
@@ -1513,9 +1513,9 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)
     }
     NEXT_BITSET_IN_SPARSEBV;
 
-    if (bailOutInfo->usedCapturedValues.argObjSyms)
+    if (bailOutInfo->usedCapturedValues->argObjSyms)
     {
-        FOREACH_BITSET_IN_SPARSEBV(id, bailOutInfo->usedCapturedValues.argObjSyms)
+        FOREACH_BITSET_IN_SPARSEBV(id, bailOutInfo->usedCapturedValues->argObjSyms)
         {
             StackSym * stackSym = this->func->m_symTable->FindStackSym(id);
             Assert(stackSym != nullptr);
@@ -1705,7 +1705,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)
                 uint outParamOffsetIndex = outParamStart + argSlot;
                 if (!sym->m_isBailOutReferenced && !sym->IsArgSlotSym())
                 {
-                    FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, constantValue, &bailOutInfo->usedCapturedValues.constantValues, iterator)
+                    FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, constantValue, &bailOutInfo->usedCapturedValues->constantValues, iterator)
                     {
                         if (constantValue.Key()->m_id == sym->m_id)
                         {
@@ -1731,13 +1731,13 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)
                         continue;
                     }
 
-                    FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSym, &bailOutInfo->usedCapturedValues.copyPropSyms, iter)
+                    FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSym, &bailOutInfo->usedCapturedValues->copyPropSyms, iter)
                     {
                         if (copyPropSym.Key()->m_id == sym->m_id)
                         {
                             StackSym * copyStackSym = copyPropSym.Value();
 
-                            BVSparse<JitArenaAllocator>* argObjSyms = bailOutInfo->usedCapturedValues.argObjSyms;
+                            BVSparse<JitArenaAllocator>* argObjSyms = bailOutInfo->usedCapturedValues->argObjSyms;
                             if (argObjSyms && argObjSyms->Test(copyStackSym->m_id))
                             {
                                 outParamOffsets[outParamOffsetIndex] = BailOutRecord::GetArgumentsObjectOffset();
@@ -1845,7 +1845,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)
                                     Assert(LowererMD::IsAssign(instrDef));
                                 }
 
-                                if (bailOutInfo->usedCapturedValues.argObjSyms && bailOutInfo->usedCapturedValues.argObjSyms->Test(sym->m_id))
+                                if (bailOutInfo->usedCapturedValues->argObjSyms && bailOutInfo->usedCapturedValues->argObjSyms->Test(sym->m_id))
                                 {
                                     //foo.apply(this,arguments) case and we bailout when the apply is overridden. We need to restore the arguments object.
                                     outParamOffsets[outParamOffsetIndex] = BailOutRecord::GetArgumentsObjectOffset();
diff --git a/lib/Backend/SccLiveness.cpp b/lib/Backend/SccLiveness.cpp
@@ -463,7 +463,7 @@ SCCLiveness::ProcessBailOutUses(IR::Instr * instr)
     }
     NEXT_BITSET_IN_SPARSEBV;
 
-    FOREACH_SLISTBASE_ENTRY(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues.copyPropSyms)
+    FOREACH_SLISTBASE_ENTRY(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues->copyPropSyms)
     {
         ProcessStackSymUse(copyPropSyms.Value(), instr);
     }

Original file line number	Diff line number	Diff line change
`@@ -1763,19 +1763,19 @@ BackwardPass::ProcessBailOutArgObj(BailOutInfo * bailOutInfo, BVSparse<JitArenaA`
`1763`	`1763`	`{`
`1764`	`1764`	`if (byteCodeUpwardExposedUsed->TestAndClear(symId))`
`1765`	`1765`	`{`
`1766`		`- if (bailOutInfo->usedCapturedValues.argObjSyms == nullptr)`
	`1766`	`+ if (bailOutInfo->usedCapturedValues->argObjSyms == nullptr)`
`1767`	`1767`	`{`
`1768`		`- bailOutInfo->usedCapturedValues.argObjSyms = JitAnew(this->func->m_alloc,`
	`1768`	`+ bailOutInfo->usedCapturedValues->argObjSyms = JitAnew(this->func->m_alloc,`
`1769`	`1769`	`BVSparse<JitArenaAllocator>, this->func->m_alloc);`
`1770`	`1770`	`}`
`1771`		`- bailOutInfo->usedCapturedValues.argObjSyms->Set(symId);`
	`1771`	`+ bailOutInfo->usedCapturedValues->argObjSyms->Set(symId);`
`1772`	`1772`	`}`
`1773`	`1773`	`}`
`1774`	`1774`	`NEXT_BITSET_IN_SPARSEBV;`
`1775`	`1775`	`}`
`1776`		`- if (bailOutInfo->usedCapturedValues.argObjSyms)`
	`1776`	`+ if (bailOutInfo->usedCapturedValues->argObjSyms)`
`1777`	`1777`	`{`
`1778`		`- byteCodeUpwardExposedUsed->Minus(bailOutInfo->usedCapturedValues.argObjSyms);`
	`1778`	`+ byteCodeUpwardExposedUsed->Minus(bailOutInfo->usedCapturedValues->argObjSyms);`
`1779`	`1779`	`}`
`1780`	`1780`	`}`
`1781`	`1781`
`@@ -1785,7 +1785,7 @@ BackwardPass::ProcessBailOutConstants(BailOutInfo * bailOutInfo, BVSparse<JitAre`
`1785`	`1785`	`Assert(this->tag != Js::BackwardPhase);`
`1786`	`1786`
`1787`	`1787`	`// Remove constants that we are already going to restore`
`1788`		`- SListBase<ConstantStackSymValue> * usedConstantValues = &bailOutInfo->usedCapturedValues.constantValues;`
	`1788`	`+ SListBase<ConstantStackSymValue> * usedConstantValues = &bailOutInfo->usedCapturedValues->constantValues;`
`1789`	`1789`	`FOREACH_SLISTBASE_ENTRY(ConstantStackSymValue, value, usedConstantValues)`
`1790`	`1790`	`{`
`1791`	`1791`	`byteCodeUpwardExposedUsed->Clear(value.Key()->m_id);`
`@@ -1817,7 +1817,7 @@ BackwardPass::ProcessBailOutCopyProps(BailOutInfo * bailOutInfo, BVSparse<JitAre`
`1817`	`1817`	`Assert(!this->func->GetJITFunctionBody()->IsAsmJsMode());`
`1818`	`1818`
`1819`	`1819`	`// Remove copy prop that we were already going to restore`
`1820`		`- SListBase<CopyPropSyms> * usedCopyPropSyms = &bailOutInfo->usedCapturedValues.copyPropSyms;`
	`1820`	`+ SListBase<CopyPropSyms> * usedCopyPropSyms = &bailOutInfo->usedCapturedValues->copyPropSyms;`
`1821`	`1821`	`FOREACH_SLISTBASE_ENTRY(CopyPropSyms, copyPropSyms, usedCopyPropSyms)`
`1822`	`1822`	`{`
`1823`	`1823`	`byteCodeUpwardExposedUsed->Clear(copyPropSyms.Key()->m_id);`
`@@ -2596,7 +2596,7 @@ BackwardPass::ProcessBailOutInfo(IR::Instr * instr, BailOutInfo * bailOutInfo)`
`2596`	`2596`	`tempBv->And(this->func->m_nonTempLocalVars, bailOutInfo->liveVarSyms);`
`2597`	`2597`
`2598`	`2598`	`// Remove syms that are restored in other ways than byteCodeUpwardExposedUsed.`
`2599`		`- FOREACH_SLIST_ENTRY(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues.constantValues)`
	`2599`	`+ FOREACH_SLIST_ENTRY(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues->constantValues)`
`2600`	`2600`	`{`
`2601`	`2601`	`Assert(value.Key()->HasByteCodeRegSlot() \|\| value.Key()->GetInstrDef()->m_opcode == Js::OpCode::BytecodeArgOutCapture);`
`2602`	`2602`	`if (value.Key()->HasByteCodeRegSlot())`
`@@ -2605,7 +2605,7 @@ BackwardPass::ProcessBailOutInfo(IR::Instr * instr, BailOutInfo * bailOutInfo)`
`2605`	`2605`	`}`
`2606`	`2606`	`}`
`2607`	`2607`	`NEXT_SLIST_ENTRY;`
`2608`		`- FOREACH_SLIST_ENTRY(CopyPropSyms, value, &bailOutInfo->usedCapturedValues.copyPropSyms)`
	`2608`	`+ FOREACH_SLIST_ENTRY(CopyPropSyms, value, &bailOutInfo->usedCapturedValues->copyPropSyms)`
`2609`	`2609`	`{`
`2610`	`2610`	`Assert(value.Key()->HasByteCodeRegSlot() \|\| value.Key()->GetInstrDef()->m_opcode == Js::OpCode::BytecodeArgOutCapture);`
`2611`	`2611`	`if (value.Key()->HasByteCodeRegSlot())`
`@@ -2614,9 +2614,9 @@ BackwardPass::ProcessBailOutInfo(IR::Instr * instr, BailOutInfo * bailOutInfo)`
`2614`	`2614`	`}`
`2615`	`2615`	`}`
`2616`	`2616`	`NEXT_SLIST_ENTRY;`
`2617`		`- if (bailOutInfo->usedCapturedValues.argObjSyms)`
	`2617`	`+ if (bailOutInfo->usedCapturedValues->argObjSyms)`
`2618`	`2618`	`{`
`2619`		`- tempBv->Minus(bailOutInfo->usedCapturedValues.argObjSyms);`
	`2619`	`+ tempBv->Minus(bailOutInfo->usedCapturedValues->argObjSyms);`
`2620`	`2620`	`}`
`2621`	`2621`
`2622`	`2622`	`byteCodeUpwardExposedUsed->Or(tempBv);`
`@@ -8226,7 +8226,7 @@ BackwardPass::ReverseCopyProp(IR::Instr *instr)`
`8226`	`8226`	`FOREACH_SLISTBASE_ENTRY(`
`8227`	`8227`	`CopyPropSyms,`
`8228`	`8228`	`usedCopyPropSym,`
`8229`		`- &instrPrev->GetBailOutInfo()->usedCapturedValues.copyPropSyms)`
	`8229`	`+ &instrPrev->GetBailOutInfo()->usedCapturedValues->copyPropSyms)`
`8230`	`8230`	`{`
`8231`	`8231`	`if(dstSym == usedCopyPropSym.Value())`
`8232`	`8232`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1417,7 +1417,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)`
`1417`	`1417`	`memset(state.registerSaveSyms, 0, sizeof(state.registerSaveSyms));`
`1418`	`1418`
`1419`	`1419`	`// Fill in the constants`
`1420`		`- FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues.constantValues, constantValuesIterator)`
	`1420`	`+ FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, value, &bailOutInfo->usedCapturedValues->constantValues, constantValuesIterator)`
`1421`	`1421`	`{`
`1422`	`1422`	`AssertMsg(bailOutInfo->bailOutRecord->bailOutKind != IR::BailOutForGeneratorYield, "constant prop syms unexpected for bail-in for generator yield");`
`1423`	`1423`	`StackSym * stackSym = value.Key();`
`@@ -1460,7 +1460,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)`
`1460`	`1460`	`NEXT_SLISTBASE_ENTRY_EDITING;`
`1461`	`1461`
`1462`	`1462`	`// Fill in the copy prop syms`
`1463`		`- FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues.copyPropSyms, copyPropSymsIter)`
	`1463`	`+ FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues->copyPropSyms, copyPropSymsIter)`
`1464`	`1464`	`{`
`1465`	`1465`	`AssertMsg(bailOutInfo->bailOutRecord->bailOutKind != IR::BailOutForGeneratorYield, "copy prop syms unexpected for bail-in for generator yield");`
`1466`	`1466`	`StackSym * stackSym = copyPropSyms.Key();`
`@@ -1513,9 +1513,9 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)`
`1513`	`1513`	`}`
`1514`	`1514`	`NEXT_BITSET_IN_SPARSEBV;`
`1515`	`1515`
`1516`		`- if (bailOutInfo->usedCapturedValues.argObjSyms)`
	`1516`	`+ if (bailOutInfo->usedCapturedValues->argObjSyms)`
`1517`	`1517`	`{`
`1518`		`- FOREACH_BITSET_IN_SPARSEBV(id, bailOutInfo->usedCapturedValues.argObjSyms)`
	`1518`	`+ FOREACH_BITSET_IN_SPARSEBV(id, bailOutInfo->usedCapturedValues->argObjSyms)`
`1519`	`1519`	`{`
`1520`	`1520`	`StackSym * stackSym = this->func->m_symTable->FindStackSym(id);`
`1521`	`1521`	`Assert(stackSym != nullptr);`
`@@ -1705,7 +1705,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)`
`1705`	`1705`	`uint outParamOffsetIndex = outParamStart + argSlot;`
`1706`	`1706`	`if (!sym->m_isBailOutReferenced && !sym->IsArgSlotSym())`
`1707`	`1707`	`{`
`1708`		`- FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, constantValue, &bailOutInfo->usedCapturedValues.constantValues, iterator)`
	`1708`	`+ FOREACH_SLISTBASE_ENTRY_EDITING(ConstantStackSymValue, constantValue, &bailOutInfo->usedCapturedValues->constantValues, iterator)`
`1709`	`1709`	`{`
`1710`	`1710`	`if (constantValue.Key()->m_id == sym->m_id)`
`1711`	`1711`	`{`
`@@ -1731,13 +1731,13 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)`
`1731`	`1731`	`continue;`
`1732`	`1732`	`}`
`1733`	`1733`
`1734`		`- FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSym, &bailOutInfo->usedCapturedValues.copyPropSyms, iter)`
	`1734`	`+ FOREACH_SLISTBASE_ENTRY_EDITING(CopyPropSyms, copyPropSym, &bailOutInfo->usedCapturedValues->copyPropSyms, iter)`
`1735`	`1735`	`{`
`1736`	`1736`	`if (copyPropSym.Key()->m_id == sym->m_id)`
`1737`	`1737`	`{`
`1738`	`1738`	`StackSym * copyStackSym = copyPropSym.Value();`
`1739`	`1739`
`1740`		`- BVSparse<JitArenaAllocator>* argObjSyms = bailOutInfo->usedCapturedValues.argObjSyms;`
	`1740`	`+ BVSparse<JitArenaAllocator>* argObjSyms = bailOutInfo->usedCapturedValues->argObjSyms;`
`1741`	`1741`	`if (argObjSyms && argObjSyms->Test(copyStackSym->m_id))`
`1742`	`1742`	`{`
`1743`	`1743`	`outParamOffsets[outParamOffsetIndex] = BailOutRecord::GetArgumentsObjectOffset();`
`@@ -1845,7 +1845,7 @@ LinearScan::FillBailOutRecord(IR::Instr * instr)`
`1845`	`1845`	`Assert(LowererMD::IsAssign(instrDef));`
`1846`	`1846`	`}`
`1847`	`1847`
`1848`		`- if (bailOutInfo->usedCapturedValues.argObjSyms && bailOutInfo->usedCapturedValues.argObjSyms->Test(sym->m_id))`
	`1848`	`+ if (bailOutInfo->usedCapturedValues->argObjSyms && bailOutInfo->usedCapturedValues->argObjSyms->Test(sym->m_id))`
`1849`	`1849`	`{`
`1850`	`1850`	`//foo.apply(this,arguments) case and we bailout when the apply is overridden. We need to restore the arguments object.`
`1851`	`1851`	`outParamOffsets[outParamOffsetIndex] = BailOutRecord::GetArgumentsObjectOffset();`
Original file line number	Diff line number	Diff line change
`@@ -463,7 +463,7 @@ SCCLiveness::ProcessBailOutUses(IR::Instr * instr)`
`463`	`463`	`}`
`464`	`464`	`NEXT_BITSET_IN_SPARSEBV;`
`465`	`465`
`466`		`- FOREACH_SLISTBASE_ENTRY(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues.copyPropSyms)`
	`466`	`+ FOREACH_SLISTBASE_ENTRY(CopyPropSyms, copyPropSyms, &bailOutInfo->usedCapturedValues->copyPropSyms)`
`467`	`467`	`{`
`468`	`468`	`ProcessStackSymUse(copyPropSyms.Value(), instr);`
`469`	`469`	`}`