Bug 156720 - Enable separated heap by default on iOS
Summary: Enable separated heap by default on iOS
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Oliver Hunt
Depends on:
Blocks: 156842
  Show dependency treegraph
Reported: 2016-04-18 16:18 PDT by Oliver Hunt
Modified: 2016-05-05 16:58 PDT (History)
9 users (show)

See Also:

Patch (1.58 KB, patch)
2016-04-18 16:23 PDT, Oliver Hunt
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Hunt 2016-04-18 16:18:43 PDT
Enable separated heap by default on ios
Comment 1 Oliver Hunt 2016-04-18 16:23:36 PDT
Created attachment 276678 [details]
Comment 2 Geoffrey Garen 2016-04-19 10:39:22 PDT
Comment on attachment 276678 [details]

Comment 3 Oliver Hunt 2016-04-19 11:33:56 PDT
Committed r199734: <http://trac.webkit.org/changeset/199734>
Comment 4 Andy Estes 2016-04-21 03:07:43 PDT
This change causes Safari in iOS Simulator to crash loading just about anything that uses JavaScript. I filed https://bugs.webkit.org/show_bug.cgi?id=156842.
Comment 5 Gavin Barraclough 2016-04-26 10:30:44 PDT
This causes a memory regression, and was rolled out in r200095.
Comment 6 Oliver Hunt 2016-04-26 10:46:26 PDT
Committed r200097

Please talk to me if before trying to roll this out again.
Comment 7 Keith Rollin 2016-05-02 12:27:26 PDT
Taking care of some paperwork:

* The change was rolled out again in http://trac.webkit.org/changeset/200134
* The underlying issue was resolved in Radar 25945386
* Once that change has landed, we can roll this change back in again.
Comment 8 Oliver Hunt 2016-05-05 16:58:35 PDT
Committed r200496: <http://trac.webkit.org/changeset/200496>