[turbofan] Make the inlining heuristic deterministic.

[turbofan] Make the inlining heuristic deterministic.

Don't sort inline candidates by comparing Node pointers, where the order
depends on the allocation order and the concrete Zone memory layout at
runtime. Instead sort based on NodeId, which is deterministic.

R=mvstanton@chromium.org
BUG=v8:5267
Committed: https://crrev.com/ae9a39b010ff74759439d3084e99a6b809085151
Cr-Commit-Position: refs/heads/master@{#39371}

[Benedikt Meurer, 2016-09-13 07:53:27]

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

[commit-bot: I haz the power, 2016-09-13 07:53:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Benedikt Meurer, 2016-09-13 07:54:06]

Hey Michael,

Here's a fix for some of the non-determinism we see with for example Richards.
Please take a look.

Thanks,
Benedikt

[mvstanton, 2016-09-13 07:58:11]

Yay predictability. lgtm. :)

[Benedikt Meurer, 2016-09-13 07:58:21]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-13 07:58:24]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-09-13 07:58:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-13 08:19:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-13 08:19:46]

Description was changed from

==========
[turbofan] Make the inlining heuristic deterministic.

Don't sort inline candidates by comparing Node pointers, where the order
depends on the allocation order and the concrete Zone memory layout at
runtime. Instead sort based on NodeId, which is deterministic.

R=mvstanton@chromium.org
BUG=v8:5267
==========

to

==========
[turbofan] Make the inlining heuristic deterministic.

Don't sort inline candidates by comparing Node pointers, where the order
depends on the allocation order and the concrete Zone memory layout at
runtime. Instead sort based on NodeId, which is deterministic.

R=mvstanton@chromium.org
BUG=v8:5267

Committed: https://crrev.com/ae9a39b010ff74759439d3084e99a6b809085151
Cr-Commit-Position: refs/heads/master@{#39371}
==========

[commit-bot: I haz the power, 2016-09-13 08:19:46]

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