Closed Bug 787288 Opened 7 years ago Closed 7 years ago

IonMonkey: Get --disable-ion configure flag working again

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dvander, Assigned: dvander)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
No description provided.
Attachment #657104 - Flags: review?(sstangl)
Comment on attachment 657104 [details] [diff] [review]
patch

Review of attachment 657104 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jscntxt.cpp
@@ +1015,5 @@
>  
>      if (rt->gcIsNeeded)
>          GCSlice(rt, GC_NORMAL, rt->gcTriggerReason);
>  
> +#ifdef JS_ION

Probably also not necessary to call resetIonStackLimit(). There's also the equivalent code in the trigger function.
Attachment #657104 - Flags: review?(sstangl) → review+
https://hg.mozilla.org/projects/ionmonkey/rev/9e38c1b2435b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.