X87: Re-reland: Remove register index/code indirection.

X87: Re-reland: Remove register index/code indirection.

    port 5cf1c0bcf6f10ff61a95a00fa302eeee7c390885 (r31087).

    original commit message:
    Previous to this patch, both the lithium and TurboFan register
    allocators tracked allocated registers by "indices", rather than
    the register codes used elsewhere in the runtime. This patch
    ensures that codes are used everywhere, and in the process cleans
    up a bunch of redundant code and adds more structure to how the
    set of allocatable registers is defined.

    Some highlights of changes:

    * TurboFan's RegisterConfiguration class moved to V8's top level
      so that it can be shared with Crankshaft.
    * Various "ToAllocationIndex" and related methods removed.
    * Code that can be easily shared between Register classes on
      different platforms is now shared.
    * The list of allocatable registers on each platform is declared
      as a list rather than implicitly via the register index <->
      code mapping.

    additional comment:
    This patch must be work with CL https://codereview.chromium.org/1405673003/
    and CL https://codereview.chromium.org/1413343002/
    which provide the needed register allocation common code change in
    v8 for this CL

BUG=

Committed: https://crrev.com/2e5845f178b2fc52d9a344f35b50d27a6d45fd03
Cr-Commit-Position: refs/heads/master@{#31494}

[zhengxing.li, 2015-10-23 07:03:51]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai@intel.com

[zhengxing.li, 2015-10-23 07:03:52]

help to review this patch, Thanks!

[zhengxing.li, 2015-10-23 07:06:29]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai;weiliang.lin@intel.com
- chunyang.dai@intel.com

[zhengxing.li, 2015-10-23 07:10:04]

Description was changed from

==========
X87: Re-reland: Remove register index/code indirection.

    port 5cf1c0bcf6f10ff61a95a00fa302eeee7c390885 (r31087).

    contributed by zhengxing.li@intel.com

    original commit message:
    Previous to this patch, both the lithium and TurboFan register
    allocators tracked allocated registers by "indices", rather than
    the register codes used elsewhere in the runtime. This patch
    ensures that codes are used everywhere, and in the process cleans
    up a bunch of redundant code and adds more structure to how the
    set of allocatable registers is defined.

    Some highlights of changes:

    * TurboFan's RegisterConfiguration class moved to V8's top level
      so that it can be shared with Crankshaft.
    * Various "ToAllocationIndex" and related methods removed.
    * Code that can be easily shared between Register classes on
      different platforms is now shared.
    * The list of allocatable registers on each platform is declared
      as a list rather than implicitly via the register index <->
      code mapping.

    additional comment:
    This patch must be work with CL https://codereview.chromium.org/1405673003/
    and CL https://codereview.chromium.org/1413343002/
    which provide the needed register allocation common code change in
    v8 for this CL

BUG=
==========

to

==========
X87: Re-reland: Remove register index/code indirection.

    port 5cf1c0bcf6f10ff61a95a00fa302eeee7c390885 (r31087).

    original commit message:
    Previous to this patch, both the lithium and TurboFan register
    allocators tracked allocated registers by "indices", rather than
    the register codes used elsewhere in the runtime. This patch
    ensures that codes are used everywhere, and in the process cleans
    up a bunch of redundant code and adds more structure to how the
    set of allocatable registers is defined.

    Some highlights of changes:

    * TurboFan's RegisterConfiguration class moved to V8's top level
      so that it can be shared with Crankshaft.
    * Various "ToAllocationIndex" and related methods removed.
    * Code that can be easily shared between Register classes on
      different platforms is now shared.
    * The list of allocatable registers on each platform is declared
      as a list rather than implicitly via the register index <->
      code mapping.

    additional comment:
    This patch must be work with CL https://codereview.chromium.org/1405673003/
    and CL https://codereview.chromium.org/1413343002/
    which provide the needed register allocation common code change in
    v8 for this CL

BUG=
==========

[Weiliang, 2015-10-23 07:11:19]

weiliang.lin@intel.com changed reviewers:
+ chunyang.dai@intel.com, weiliang.lin@intel.com
- chunyang.dai;weiliang.lin@intel.com

[Weiliang, 2015-10-23 07:11:19]

The CQ bit was checked by weiliang.lin@intel.com

[Weiliang, 2015-10-23 07:11:19]

lgtm

[zhengxing.li, 2015-10-23 07:26:34]

Patchset #2 (id:20001) has been deleted

[zhengxing.li, 2015-10-23 07:57:11]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2015-10-23 07:57:15]

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

[commit-bot: I haz the power, 2015-10-23 07:58:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-23 07:58:57]

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