[MERGE #5882 @pleath] Turn LazyBailout off by default

pleath · pleath · commit 720e28963997 · 2018-12-17T16:03:20.000-08:00
Merge pull request #5882 from pleath:lboff The main phase switch was flipped the wrong way.
diff --git a/lib/Backend/Func.cpp b/lib/Backend/Func.cpp
@@ -2111,7 +2111,7 @@ bool
 Func::ShouldDoLazyBailOut() const
 {
 #if defined(_M_X64)
-    if (PHASE_ON1(Js::LazyBailoutPhase) ||
+    if (!PHASE_ON1(Js::LazyBailoutPhase) ||
         this->GetJITFunctionBody()->IsAsmJsMode() || // don't have bailouts in asm.js
         this->HasTry() ||                            // lazy bailout in function with try/catch not supported for now
                                                      // `EHBailoutPatchUp` set a `hasBailedOut` bit to rethrow the exception in the interpreter

Original file line number	Diff line number	Diff line change
`@@ -2111,7 +2111,7 @@ bool`
`2111`	`2111`	`Func::ShouldDoLazyBailOut() const`
`2112`	`2112`	`{`
`2113`	`2113`	`#if defined(_M_X64)`
`2114`		`- if (PHASE_ON1(Js::LazyBailoutPhase) \|\|`
	`2114`	`+ if (!PHASE_ON1(Js::LazyBailoutPhase) \|\|`
`2115`	`2115`	`this->GetJITFunctionBody()->IsAsmJsMode() \|\| // don't have bailouts in asm.js`
`2116`	`2116`	`this->HasTry() \|\| // lazy bailout in function with try/catch not supported for now`
`2117`	`2117`	// `EHBailoutPatchUp` set a `hasBailedOut` bit to rethrow the exception in the interpreter