Move SourcePosition into separate header file.

Move SourcePosition into separate header file.

This splits out the SourcePosition class into a separate header file.
Reason for this refactoring is that said class is mostly used by the
Crankshaft compiler and not needed for all compilers. Also having the
assembler depend on the class creates a dependency cycle.

R=bmeurer@chromium.org

Committed: https://crrev.com/0aeaf0cbd1427d063fbf0298590ab8a91e6c7221
Cr-Commit-Position: refs/heads/master@{#33325}

[Michael Starzinger, 2016-01-15 10:54:39]

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

[commit-bot: I haz the power, 2016-01-15 10:54:49]

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

[commit-bot: I haz the power, 2016-01-15 11:38:20]

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

[commit-bot: I haz the power, 2016-01-15 11:38:21]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-01-15 11:38:49]

LGTM once comment is addressed.

https://codereview.chromium.org/1581083009/diff/1/src/source-position.h
File src/source-position.h (right):

https://codereview.chromium.org/1581083009/diff/1/src/source-position.h#newcode6
src/source-position.h:6: #define V8_SOURCE_POSITION_H_
Can you include the necessary headers here?

[Michael Starzinger, 2016-01-15 11:46:03]

https://codereview.chromium.org/1581083009/diff/1/src/source-position.h
File src/source-position.h (right):

https://codereview.chromium.org/1581083009/diff/1/src/source-position.h#newcode6
src/source-position.h:6: #define V8_SOURCE_POSITION_H_
On 2016/01/15 11:38:49, Benedikt Meurer wrote:
> Can you include the necessary headers here?

Done. Ooops, oversight.

[Michael Starzinger, 2016-01-15 11:47:00]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-01-15 11:47:00]

The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/1581083009/#ps20001
(title: "Include what you use.")

[commit-bot: I haz the power, 2016-01-15 11:47:03]

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

[commit-bot: I haz the power, 2016-01-15 12:11:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-15 12:11:57]

Description was changed from

==========
Move SourcePosition into separate header file.

This splits out the SourcePosition class into a separate header file.
Reason for this refactoring is that said class is mostly used by the
Crankshaft compiler and not needed for all compilers. Also having the
assembler depend on the class creates a dependency cycle.

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

to

==========
Move SourcePosition into separate header file.

This splits out the SourcePosition class into a separate header file.
Reason for this refactoring is that said class is mostly used by the
Crankshaft compiler and not needed for all compilers. Also having the
assembler depend on the class creates a dependency cycle.

R=bmeurer@chromium.org

Committed: https://crrev.com/0aeaf0cbd1427d063fbf0298590ab8a91e6c7221
Cr-Commit-Position: refs/heads/master@{#33325}
==========

[commit-bot: I haz the power, 2016-01-15 12:11:58]

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