[turbofan] split register allocator into little pieces

[turbofan] split register allocator into little pieces

R=titzer@chromium.org

BUG=

Committed: https://crrev.com/497c537310d72064a23bb0ea3e4e708460b7dbf4
Cr-Commit-Position: refs/heads/master@{#27946}

[dcarney, 2015-04-20 13:09:33]

dcarney@chromium.org changed reviewers:
+ titzer@chromium.org

[titzer, 2015-04-20 13:33:53]

Looks good, just one more slice.

https://codereview.chromium.org/1094063002/diff/100001/src/compiler/register-...
File src/compiler/register-allocator.h (right):

https://codereview.chromium.org/1094063002/diff/100001/src/compiler/register-...
src/compiler/register-allocator.h:533: const RegisterConfiguration* const
config_;
Can we split the output of the LiveRangeBuilder into a separate data-only class
(hopefully struct!) so that we can pass that around?

[dcarney, 2015-04-20 14:20:44]

On 2015/04/20 13:33:53, titzer wrote:
> Looks good, just one more slice.
> 
>
https://codereview.chromium.org/1094063002/diff/100001/src/compiler/register-...
> File src/compiler/register-allocator.h (right):
> 
>
https://codereview.chromium.org/1094063002/diff/100001/src/compiler/register-...
> src/compiler/register-allocator.h:533: const RegisterConfiguration* const
> config_;
> Can we split the output of the LiveRangeBuilder into a separate data-only
class
> (hopefully struct!) so that we can pass that around?

done.

a struct? are you kidding?

[titzer, 2015-04-20 14:54:37]

LGTM although a struct would be soooo much nicer

[dcarney, 2015-04-20 15:33:42]

The CQ bit was checked by dcarney@chromium.org

[commit-bot: I haz the power, 2015-04-20 15:33:51]

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

[commit-bot: I haz the power, 2015-04-20 16:15:40]

Committed patchset #9 (id:150001)

[commit-bot: I haz the power, 2015-04-20 16:15:50]

Patchset 9 (id:??) landed as
https://crrev.com/497c537310d72064a23bb0ea3e4e708460b7dbf4
Cr-Commit-Position: refs/heads/master@{#27946}