Assert for rejitCounter

yullin-ms · yullin-ms · commit c89ab41e56e7 · 2018-09-12T12:26:54.000-07:00
diff --git a/lib/Backend/Func.cpp b/lib/Backend/Func.cpp
@@ -303,8 +303,10 @@ Func::Codegen(JitArenaAllocator *alloc, JITTimeWorkItem * workItem,
     Js::ScriptContextProfiler *const codeGenProfiler, const bool isBackgroundJIT)
 {
     bool rejit;
+    int rejitCounter = 0;
     do
     {
+        Assert(rejitCounter < 25);
         Func func(alloc, workItem, threadContextInfo,
             scriptContextInfo, outputData, epInfo, runtimeInfo,
             polymorphicInlineCacheInfo, codeGenAllocators, 
@@ -364,6 +366,7 @@ Func::Codegen(JitArenaAllocator *alloc, JITTimeWorkItem * workItem,
             }
 
             rejit = true;
+            rejitCounter++;
         }
         // Either the entry point has a reference to the number now, or we failed to code gen and we
         // don't need to numbers, we can flush the completed page now.

Original file line number	Diff line number	Diff line change
`@@ -303,8 +303,10 @@ Func::Codegen(JitArenaAllocator alloc, JITTimeWorkItem workItem,`
`303`	`303`	`Js::ScriptContextProfiler *const codeGenProfiler, const bool isBackgroundJIT)`
`304`	`304`	`{`
`305`	`305`	`bool rejit;`
	`306`	`+ int rejitCounter = 0;`
`306`	`307`	`do`
`307`	`308`	`{`
	`309`	`+ Assert(rejitCounter < 25);`
`308`	`310`	`Func func(alloc, workItem, threadContextInfo,`
`309`	`311`	`scriptContextInfo, outputData, epInfo, runtimeInfo,`
`310`	`312`	`polymorphicInlineCacheInfo, codeGenAllocators,`
`@@ -364,6 +366,7 @@ Func::Codegen(JitArenaAllocator alloc, JITTimeWorkItem workItem,`
`364`	`366`	`}`
`365`	`367`
`366`	`368`	`rejit = true;`
	`369`	`+ rejitCounter++;`
`367`	`370`	`}`
`368`	`371`	`// Either the entry point has a reference to the number now, or we failed to code gen and we`
`369`	`372`	`// don't need to numbers, we can flush the completed page now.`