Reland Store OSR'd optimized code on the native context.

Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.

Since we OSR code rarely, it makes sense to store it and look for it on the native context rather than the SharedFunctionInfo. This makes the OptimizedCodeMap data structure more space efficient, as it doesn't have to store an ast ID for the OSR entry point.

Committed: https://crrev.com/044b2d1bd888dc19753f3717f362f90944614cc7
Cr-Commit-Position: refs/heads/master@{#41584}

[mvstanton, 2016-12-08 10:40:19]

Description was changed from

==========
Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.
==========

to

==========
Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.

Since we OSR code rarely, it makes sense to store it and look for it on the
native context rather than the SharedFunctionInfo. This makes the
OptimizedCodeMap data structure more space efficient, as it doesn't have to
store an ast ID for the OSR entry point.
==========

[mvstanton, 2016-12-08 10:40:19]

mvstanton@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[mvstanton, 2016-12-08 10:41:26]

Hi Michael,
Here is the reland of yesterday's CL with the bugfix in objects.cc included.
Thanks for the look!

[Michael Starzinger, 2016-12-08 10:42:34]

LGTM (rubber-stamp, assuming it is a mechanical combination of the two CLs that
I already reviewed).

[mvstanton, 2016-12-08 10:45:49]

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

[commit-bot: I haz the power, 2016-12-08 10:45:51]

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

[mvstanton, 2016-12-08 10:46:51]

The CQ bit was checked by mvstanton@chromium.org

[commit-bot: I haz the power, 2016-12-08 10:47:02]

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-12-08 11:13:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481194011168390, "parent_rev":
"86869d8d5b9098ab960911324896575ae0147edb", "commit_rev":
"233d4a371c37638479d0f3ffd24d987986234be5"}

[commit-bot: I haz the power, 2016-12-08 11:13:35]

Description was changed from

==========
Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.

Since we OSR code rarely, it makes sense to store it and look for it on the
native context rather than the SharedFunctionInfo. This makes the
OptimizedCodeMap data structure more space efficient, as it doesn't have to
store an ast ID for the OSR entry point.
==========

to

==========
Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.

Since we OSR code rarely, it makes sense to store it and look for it on the
native context rather than the SharedFunctionInfo. This makes the
OptimizedCodeMap data structure more space efficient, as it doesn't have to
store an ast ID for the OSR entry point.
==========

[commit-bot: I haz the power, 2016-12-08 11:13:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-08 11:14:05]

Description was changed from

==========
Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.

Since we OSR code rarely, it makes sense to store it and look for it on the
native context rather than the SharedFunctionInfo. This makes the
OptimizedCodeMap data structure more space efficient, as it doesn't have to
store an ast ID for the OSR entry point.
==========

to

==========
Reland Store OSR'd optimized code on the native context.

The patch was reverted due to a bug - we failed to evict OSR-optimized
code in the case where the SharedFunctionInfo OptimizedCodeMap was
empty/cleared.

Since we OSR code rarely, it makes sense to store it and look for it on the
native context rather than the SharedFunctionInfo. This makes the
OptimizedCodeMap data structure more space efficient, as it doesn't have to
store an ast ID for the OSR entry point.

Committed: https://crrev.com/044b2d1bd888dc19753f3717f362f90944614cc7
Cr-Commit-Position: refs/heads/master@{#41584}
==========

[commit-bot: I haz the power, 2016-12-08 11:14:06]

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