Don't include ICData::owner_ in precompiled snapshots.

Don't include ICData::owner_ in precompiled snapshots.

It can refer to a function that has always been inlined and that the
tree shaker has removed.

precompiled dart2js x64 21647652 -> 21437594 (-0.97%)

R=srdjan@google.com

Committed: https://github.com/dart-lang/sdk/commit/c03761a21adf348b14c3607c16c4842cd256ff2c

[rmacnak, 2016-02-16 20:16:24]

rmacnak@google.com changed reviewers:
+ srdjan@google.com

[srdjan, 2016-02-16 20:36:43]

lgtm

https://codereview.chromium.org/1698323003/diff/20001/runtime/vm/raw_object_s...
File runtime/vm/raw_object_snapshot.cc (right):

https://codereview.chromium.org/1698323003/diff/20001/runtime/vm/raw_object_s...
runtime/vm/raw_object_snapshot.cc:2030: // refer to a function which was always
inlined and no longer need.
s/need/needed/

[rmacnak, 2016-02-16 21:38:32]

Description was changed from

==========
Don't include ICData::owner_ in precompiled snapshots.

It can refer to a function that has always been inlined and that the
tree shaker has removed.

precompiled dart2js x64 21647652 -> 21437594 (-0.97%)
==========

to

==========
Don't include ICData::owner_ in precompiled snapshots.

It can refer to a function that has always been inlined and that the
tree shaker has removed.

precompiled dart2js x64 21647652 -> 21437594 (-0.97%)

R=srdjan@google.com

Committed:
https://github.com/dart-lang/sdk/commit/c03761a21adf348b14c3607c16c4842cd256ff2c
==========

[rmacnak, 2016-02-16 21:38:33]

Committed patchset #2 (id:20001) manually as
c03761a21adf348b14c3607c16c4842cd256ff2c (presubmit successful).

[rmacnak, 2016-02-16 21:39:17]

https://codereview.chromium.org/1698323003/diff/20001/runtime/vm/raw_object_s...
File runtime/vm/raw_object_snapshot.cc (right):

https://codereview.chromium.org/1698323003/diff/20001/runtime/vm/raw_object_s...
runtime/vm/raw_object_snapshot.cc:2030: // refer to a function which was always
inlined and no longer need.
On 2016/02/16 20:36:42, srdjan wrote:
> s/need/needed/

Done.