Separate greedy regalloc in its own files

Separated the new register allocator in its own files.

Opportunistically removed GreedyAllocator::TryReuseSpillForPhi because it is actually unsuitable for Greedy. It was copied from Linear and it relies on hints, however, the current implementation of hints assumes linear scan.

This change doesn't aim to address performance nor correctness for Greedy.

BUG=

Committed: https://crrev.com/4e0b7b0cf5c591219ad1152a832123a9837eeecb
Cr-Commit-Position: refs/heads/master@{#29054}

[Mircea Trofin, 2015-06-15 21:40:16]

mtrofin@chromium.org changed reviewers:
+ bmeurer@chromium.org, jarin@chromium.org

[Mircea Trofin, 2015-06-15 21:55:28]

https://codereview.chromium.org/1184183002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1184183002/diff/1/BUILD.gn#newcode650
BUILD.gn:650: "src/compiler/greedy-allocator.h",
I am actually not confident this change needs to be done in this file. The build
was unblocked when I modified the gyp file in this change.

What is the purpose of this file?

[Benedikt Meurer, 2015-06-16 05:32:01]

https://codereview.chromium.org/1184183002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1184183002/diff/1/BUILD.gn#newcode650
BUILD.gn:650: "src/compiler/greedy-allocator.h",
On 2015/06/15 21:55:28, Mircea Trofin wrote:
> I am actually not confident this change needs to be done in this file. The
build
> was unblocked when I modified the gyp file in this change.
> 
> What is the purpose of this file?

GN is the new GYP (in some unknown, distant future, in a galaxy far far away...)

[Jarin, 2015-06-16 06:39:30]

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

[Jarin, 2015-06-16 06:39:41]

lgm with a nit.

https://codereview.chromium.org/1184183002/diff/1/src/compiler/greedy-allocat...
File src/compiler/greedy-allocator.h (right):

https://codereview.chromium.org/1184183002/diff/1/src/compiler/greedy-allocat...
src/compiler/greedy-allocator.h:63: }
Missing comments for namespace endings.

[commit-bot: I haz the power, 2015-06-16 06:39:41]

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

[Jarin, 2015-06-16 06:39:59]

I mean lgtm :-)

[commit-bot: I haz the power, 2015-06-16 07:08:36]

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

[commit-bot: I haz the power, 2015-06-16 07:08:37]

Dry run: This issue passed the CQ dry run.

[Mircea Trofin, 2015-06-16 16:16:11]

The CQ bit was checked by mtrofin@chromium.org

[Mircea Trofin, 2015-06-16 16:16:11]

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

[commit-bot: I haz the power, 2015-06-16 16:16:17]

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

[Mircea Trofin, 2015-06-16 16:16:25]

https://codereview.chromium.org/1184183002/diff/1/src/compiler/greedy-allocat...
File src/compiler/greedy-allocator.h (right):

https://codereview.chromium.org/1184183002/diff/1/src/compiler/greedy-allocat...
src/compiler/greedy-allocator.h:63: }
On 2015/06/16 06:39:40, jarin wrote:
> Missing comments for namespace endings.

Done.

[commit-bot: I haz the power, 2015-06-16 17:10:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-16 17:10:41]

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