[turbolizer] Fully parse schedule data.

[turbolizer] Fully parse schedule data.

Fixes bug 5128 in product V8.

R=danno@chromium.org
BUG=

Committed: https://crrev.com/f56793071270687b9d25880330af8b777aa6346d
Cr-Commit-Position: refs/heads/master@{#37146}

[bgeron, 2016-06-21 13:48:25]

Please take a look.

Our current --trace-turbo output crashes the schedule parsing in Turbolizer,
because it contains an empty parameter: "TypedStateValues[]".

In an attempt to attach more information, I created an associated issue in
Monorail; I'm not sure if that will be resolved automatically by landing this
patch.

The backslash before the close bracket is not strictly required I think (as
we're not inside a [] character list), but to me it seemed good style at the
time.

[bgeron, 2016-06-21 13:49:48]

Description was changed from

==========
[turbolizer] Fully parse schedule data.

Fixes bug 5128.

R=danno@chromium.org
BUG=5128
==========

to

==========
[turbolizer] Fully parse schedule data.

Fixes bug 5128 in product V8.

R=danno@chromium.org
BUG=
==========

[bgeron, 2016-06-21 13:50:42]

On 2016/06/21 13:48:25, bgeron wrote:
> Please take a look.
> 
> Our current --trace-turbo output crashes the schedule parsing in Turbolizer,
> because it contains an empty parameter: "TypedStateValues[]".
> 
> In an attempt to attach more information, I created an associated issue in
> Monorail; I'm not sure if that will be resolved automatically by landing this
> patch.
> 
> The backslash before the close bracket is not strictly required I think (as
> we're not inside a [] character list), but to me it seemed good style at the
> time.

The associated Monorail issue is at
https://bugs.chromium.org/p/v8/issues/detail?id=5128 .

[danno, 2016-06-21 14:36:08]

lgtm

[bgeron, 2016-06-21 14:56:45]

The CQ bit was checked by bgeron@google.com

[commit-bot: I haz the power, 2016-06-21 14:56:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2087813002/1

[commit-bot: I haz the power, 2016-06-21 15:18:08]

Description was changed from

==========
[turbolizer] Fully parse schedule data.

Fixes bug 5128 in product V8.

R=danno@chromium.org
BUG=
==========

to

==========
[turbolizer] Fully parse schedule data.

Fixes bug 5128 in product V8.

R=danno@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-06-21 15:18:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-21 15:18:52]

Description was changed from

==========
[turbolizer] Fully parse schedule data.

Fixes bug 5128 in product V8.

R=danno@chromium.org
BUG=
==========

to

==========
[turbolizer] Fully parse schedule data.

Fixes bug 5128 in product V8.

R=danno@chromium.org
BUG=

Committed: https://crrev.com/f56793071270687b9d25880330af8b777aa6346d
Cr-Commit-Position: refs/heads/master@{#37146}
==========

[commit-bot: I haz the power, 2016-06-21 15:18:53]

Patchset 1 (id:??) landed as
https://crrev.com/f56793071270687b9d25880330af8b777aa6346d
Cr-Commit-Position: refs/heads/master@{#37146}