Parallelize test/mjsunit/es6/tail-call-megatest.js in order to avoid timeouts on slow architectures.

Parallelize test/mjsunit/es6/tail-call-megatest.js in order to avoid timeouts on slow architectures.

Committed: https://crrev.com/209eb337b17590a14f9a9d2d5d6c620f5c2e1d24
Cr-Commit-Position: refs/heads/master@{#34622}

[Igor Sheludko, 2016-03-09 13:10:55]

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

[commit-bot: I haz the power, 2016-03-09 13:10:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1778663004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1778663004/1

[Igor Sheludko, 2016-03-09 13:11:20]

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

[Igor Sheludko, 2016-03-09 13:11:21]

PTAL

[commit-bot: I haz the power, 2016-03-09 13:33:44]

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

[commit-bot: I haz the power, 2016-03-09 13:33:45]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-03-09 13:37:05]

LGTM with one suggestion.

https://codereview.chromium.org/1778663004/diff/1/test/mjsunit/es6/tail-call-...
File test/mjsunit/es6/tail-call-megatest.js (right):

https://codereview.chromium.org/1778663004/diff/1/test/mjsunit/es6/tail-call-...
test/mjsunit/es6/tail-call-megatest.js:42: var SHARDS_COUNT = 10;
suggestion: Instead of hard-coding the shard-count here, we could pass it in as
a second argument to "run-tests" below, thereby the entire decision of how to
shard the tests would be in the wrapper scripts and this script would be
agnostic to the total number of wrappers. Just a suggestion, up to you.

[Michael Starzinger, 2016-03-09 13:38:26]

Also, it might be better to remove the "all" test case and just shard on all
configurations (also the fast ones), otherwise the fast ones will be punishes by
having to run the entire test twice essentially.

[Igor Sheludko, 2016-03-09 13:43:12]

I removed the all test. Thanks.

https://codereview.chromium.org/1778663004/diff/1/test/mjsunit/es6/tail-call-...
File test/mjsunit/es6/tail-call-megatest.js (right):

https://codereview.chromium.org/1778663004/diff/1/test/mjsunit/es6/tail-call-...
test/mjsunit/es6/tail-call-megatest.js:42: var SHARDS_COUNT = 10;
On 2016/03/09 13:37:05, Michael Starzinger wrote:
> suggestion: Instead of hard-coding the shard-count here, we could pass it in
as
> a second argument to "run-tests" below, thereby the entire decision of how to
> shard the tests would be in the wrapper scripts and this script would be
> agnostic to the total number of wrappers. Just a suggestion, up to you.

On the other hand the addition of new shards will require to update shard count
in all existing shard runners. I'd prefer to keep it like this.

[Igor Sheludko, 2016-03-09 13:43:36]

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

[commit-bot: I haz the power, 2016-03-09 13:43:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1778663004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1778663004/20001

[Igor Sheludko, 2016-03-09 13:57:16]

The CQ bit was unchecked by ishell@chromium.org

[Igor Sheludko, 2016-03-09 13:57:20]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2016-03-09 13:57:21]

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/1778663004/#ps20001
(title: "Addressing comments")

[commit-bot: I haz the power, 2016-03-09 13:57:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1778663004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1778663004/20001

[Igor Sheludko, 2016-03-09 14:02:46]

The CQ bit was unchecked by ishell@chromium.org

[Igor Sheludko, 2016-03-09 14:05:49]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-03-09 14:05:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1778663004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1778663004/20001

[commit-bot: I haz the power, 2016-03-09 14:07:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-09 14:09:06]

Description was changed from

==========
Parallelize test/mjsunit/es6/tail-call-megatest.js in order to avoid timeouts on
slow architectures.
==========

to

==========
Parallelize test/mjsunit/es6/tail-call-megatest.js in order to avoid timeouts on
slow architectures.

Committed: https://crrev.com/209eb337b17590a14f9a9d2d5d6c620f5c2e1d24
Cr-Commit-Position: refs/heads/master@{#34622}
==========

[commit-bot: I haz the power, 2016-03-09 14:09:07]

Patchset 2 (id:??) landed as
https://crrev.com/209eb337b17590a14f9a9d2d5d6c620f5c2e1d24
Cr-Commit-Position: refs/heads/master@{#34622}