[wasm] Move protected instruction info to RelocInfo

[wasm] Move protected instruction info to RelocInfo

Previously this information was encoded in a FixedArray dangling off the
Code object. This extra field seems to be responsible for increased memory
usage, as seen in the linked bugs. In this change, we instead encode this
in the RelocInfo and remove the field from the Code object.

BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=678583
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=671180
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=670733

Review-Url: https://codereview.chromium.org/2651833003
Cr-Commit-Position: refs/heads/master@{#42802}
Committed: https://chromium.googlesource.com/v8/v8/+/91f8a063cc508f51a18ad9c69faf032552c9d012

[Eric Holk, 2017-01-23 23:42:08]

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

[commit-bot: I haz the power, 2017-01-23 23:42:15]

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

[Eric Holk, 2017-01-23 23:42:51]

eholk@chromium.org changed reviewers:
+ ahaas@chromium.org, bradnelson@chromium.org, mtrofin@chromium.org,
titzer@chromium.org

[commit-bot: I haz the power, 2017-01-24 00:18:29]

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

[commit-bot: I haz the power, 2017-01-24 00:18:29]

Dry run: This issue passed the CQ dry run.

[titzer, 2017-01-31 01:52:19]

On 2017/01/23 23:42:52, Eric Holk wrote:

I like this simplification! LGTM

Sorry for the delay.

[Eric Holk, 2017-01-31 01:57:14]

The CQ bit was checked by eholk@chromium.org

[commit-bot: I haz the power, 2017-01-31 01:57:19]

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, 2017-01-31 02:25:58]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485827834579640,
"parent_rev": "bc5708179522babd871012c0d1dd0f64609ae71a", "commit_rev":
"91f8a063cc508f51a18ad9c69faf032552c9d012"}

[commit-bot: I haz the power, 2017-01-31 02:26:03]

Description was changed from

==========
[wasm] Move protected instruction info to RelocInfo

Previously this information was encoded in a FixedArray dangling off the
Code object. This extra field seems to be responsible for increased memory
usage, as seen in the linked bugs. In this change, we instead encode this
in the RelocInfo and remove the field from the Code object.

BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=678583
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=671180
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=670733
==========

to

==========
[wasm] Move protected instruction info to RelocInfo

Previously this information was encoded in a FixedArray dangling off the
Code object. This extra field seems to be responsible for increased memory
usage, as seen in the linked bugs. In this change, we instead encode this
in the RelocInfo and remove the field from the Code object.

BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=678583
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=671180
BUG= https://bugs.chromium.org/p/chromium/issues/detail?id=670733

Review-Url: https://codereview.chromium.org/2651833003
Cr-Commit-Position: refs/heads/master@{#42802}
Committed:
https://chromium.googlesource.com/v8/v8/+/91f8a063cc508f51a18ad9c69faf032552c...
==========

[commit-bot: I haz the power, 2017-01-31 02:26:04]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/91f8a063cc508f51a18ad9c69faf032552c...