[MERGE chakra-core#1114] Remove bad assert.

LouisLaf · LouisLaf · commit 59a7b54b143b · 2016-06-09T16:52:05.000-07:00
Merge pull request chakra-core#1114 from LouisLaf:rejit Fix for 7832185. My rejit changes added and assert which I then made invalid. The assert said the code path can only hit once which isn't true anymore.
diff --git a/lib/Backend/BailOut.cpp b/lib/Backend/BailOut.cpp
@@ -1877,7 +1877,6 @@ void BailOutRecord::ScheduleFunctionCodeGen(Js::ScriptFunction * function, Js::S
                 if ((executeFunction->GetSavedImplicitCallsFlags() & savedImplicitCallFlags) == Js::ImplicitCall_None)
                 {
                     profileInfo->RecordImplicitCallFlags(savedImplicitCallFlags);
-                    Assert(!profileInfo->IsLoopImplicitCallInfoDisabled());
                     profileInfo->DisableLoopImplicitCallInfo();
                     rejitReason = RejitReason::ImplicitCallFlagsChanged;
                 }

Original file line number	Diff line number	Diff line change
`@@ -1877,7 +1877,6 @@ void BailOutRecord::ScheduleFunctionCodeGen(Js::ScriptFunction * function, Js::S`
`1877`	`1877`	`if ((executeFunction->GetSavedImplicitCallsFlags() & savedImplicitCallFlags) == Js::ImplicitCall_None)`
`1878`	`1878`	`{`
`1879`	`1879`	`profileInfo->RecordImplicitCallFlags(savedImplicitCallFlags);`
`1880`		`- Assert(!profileInfo->IsLoopImplicitCallInfoDisabled());`
`1881`	`1880`	`profileInfo->DisableLoopImplicitCallInfo();`
`1882`	`1881`	`rejitReason = RejitReason::ImplicitCallFlagsChanged;`
`1883`	`1882`	`}`