[turbofan] Allocate SourcePositionTable inside a Zone.

[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=bmeurer@chromium.org

Committed: https://crrev.com/8e03bdefc041c03b3028a6b5a50ce791c3b8dbbd
Cr-Commit-Position: refs/heads/master@{#35732}

[Michael Starzinger, 2016-04-22 12:46:29]

Description was changed from

==========
[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=
==========

to

==========
[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=bmeurer@chromium.org
==========

[Michael Starzinger, 2016-04-22 12:46:29]

mstarzinger@chromium.org changed reviewers:
+ bmeurer@chromium.org, clemensh@chromium.org

[Michael Starzinger, 2016-04-22 12:46:48]

Benedikt: PTAL.
Clemens: FYI.

[Jarin, 2016-04-22 13:47:58]

jarin@chromium.org changed reviewers:
+ jarin@chromium.org

[Jarin, 2016-04-22 13:47:59]

lgtm

[Michael Starzinger, 2016-04-22 13:54:39]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-04-22 13:54:45]

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

[commit-bot: I haz the power, 2016-04-22 13:57:07]

Description was changed from

==========
[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=bmeurer@chromium.org
==========

to

==========
[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=bmeurer@chromium.org
==========

[commit-bot: I haz the power, 2016-04-22 13:57:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:15:42]

Description was changed from

==========
[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=bmeurer@chromium.org
==========

to

==========
[turbofan] Allocate SourcePositionTable inside a Zone.

This also removes the destructor of the class in question, which removed
any added decorator from the graph. However the adding of the decorator
happens explicitly, so symmetry suggests that removal should also happen
explicitly instead of implicitly in the destructor.

R=bmeurer@chromium.org

Committed: https://crrev.com/8e03bdefc041c03b3028a6b5a50ce791c3b8dbbd
Cr-Commit-Position: refs/heads/master@{#35732}
==========

[commit-bot: I haz the power, 2016-04-22 19:15:43]

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