[Platform] Increase default stack size on Mac OS X to 1MB.

[Platform] Increase default stack size on Mac OS X to 1MB.

The default stack size of a background thread is 512KB on MacOSX. We default to
1MB stack checks when compiling JS code, so we need to increase this limit
to enable compilation of JS code onto background threads.

Corresponding Chromium CL is https://codereview.chromium.org/2640803002/

BUG=v8:5203
Review-Url: https://codereview.chromium.org/2653673007
Cr-Commit-Position: refs/heads/master@{#42650}
Committed: https://chromium.googlesource.com/v8/v8/+/8bae36f963a20dad603819bd5901fcf90756d78a

[rmcilroy, 2017-01-24 14:58:45]

The CQ bit was checked by rmcilroy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-24 14:58:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[rmcilroy, 2017-01-24 15:00:41]

rmcilroy@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[rmcilroy, 2017-01-24 15:00:41]

Michi could you please take a look, thanks.

[commit-bot: I haz the power, 2017-01-24 15:35:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-24 15:35:00]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2017-01-25 09:26:22]

LGTM.

https://codereview.chromium.org/2653673007/diff/1/src/base/platform/platform-...
File src/base/platform/platform-posix.cc (right):

https://codereview.chromium.org/2653673007/diff/1/src/base/platform/platform-...
src/base/platform/platform-posix.cc:625: // Default on AIX is 512kB -- bump up
to 1MB
nit: Comment is off, s/AIX/MacOS/ here.

[rmcilroy, 2017-01-25 10:03:03]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2017-01-25 10:03:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2653673007/#ps20001
(title: "Fix comment")

[commit-bot: I haz the power, 2017-01-25 10:03:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-25 10:35:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485338583440620,
"parent_rev": "a4183e59a7db2dc241070c8f02d6cb6f6545b23d", "commit_rev":
"8bae36f963a20dad603819bd5901fcf90756d78a"}

[commit-bot: I haz the power, 2017-01-25 10:35:17]

Description was changed from

==========
[Platform] Increase default stack size on Mac OS X to 1MB.

The default stack size of a background thread is 512KB on MacOSX. We default to
1MB stack checks when compiling JS code, so we need to increase this limit
to enable compilation of JS code onto background threads.

Corresponding Chromium CL is https://codereview.chromium.org/2640803002/

BUG=v8:5203
==========

to

==========
[Platform] Increase default stack size on Mac OS X to 1MB.

The default stack size of a background thread is 512KB on MacOSX. We default to
1MB stack checks when compiling JS code, so we need to increase this limit
to enable compilation of JS code onto background threads.

Corresponding Chromium CL is https://codereview.chromium.org/2640803002/

BUG=v8:5203

Review-Url: https://codereview.chromium.org/2653673007
Cr-Commit-Position: refs/heads/master@{#42650}
Committed:
https://chromium.googlesource.com/v8/v8/+/8bae36f963a20dad603819bd5901fcf9075...
==========

[commit-bot: I haz the power, 2017-01-25 10:35:18]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/8bae36f963a20dad603819bd5901fcf9075...

[rmcilroy, 2017-01-25 11:05:03]

https://codereview.chromium.org/2653673007/diff/1/src/base/platform/platform-...
File src/base/platform/platform-posix.cc (right):

https://codereview.chromium.org/2653673007/diff/1/src/base/platform/platform-...
src/base/platform/platform-posix.cc:625: // Default on AIX is 512kB -- bump up
to 1MB
On 2017/01/25 09:26:22, Michael Starzinger wrote:
> nit: Comment is off, s/AIX/MacOS/ here.

Done.