Serialize ResolutionImpact instead of WorldImpact.

Serialize ResolutionImpact instead of WorldImpact.

R=sigmund@google.com

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

[Johnni Winther, 2016-03-30 11:56:40]

johnniwinther@google.com changed reviewers:
+ sigmund@google.com

[Siggi Cherem (dart-lang), 2016-03-30 20:29:39]

lgtm

https://codereview.chromium.org/1839243003/diff/1/pkg/compiler/lib/src/compil...
File pkg/compiler/lib/src/compiler.dart (right):

https://codereview.chromium.org/1839243003/diff/1/pkg/compiler/lib/src/compil...
pkg/compiler/lib/src/compiler.dart:2029: if
(compiler.serialization.supportSerialization) {
why do we only cache when serialization is enabled?

[Johnni Winther, 2016-03-31 07:50:41]

https://codereview.chromium.org/1839243003/diff/1/pkg/compiler/lib/src/compil...
File pkg/compiler/lib/src/compiler.dart (right):

https://codereview.chromium.org/1839243003/diff/1/pkg/compiler/lib/src/compil...
pkg/compiler/lib/src/compiler.dart:2029: if
(compiler.serialization.supportSerialization) {
On 2016/03/30 20:29:39, Siggi Cherem (dart-lang) wrote:
> why do we only cache when serialization is enabled?

We always the the WorldImpact, which is the one use by the enqueuer. The
ResolutionImpact is (currently) only used in itself in serialization.

Adding a TODO to align these use cases better.

[Johnni Winther, 2016-03-31 07:55:12]

Description was changed from

==========
Serialize ResolutionImpact instead of WorldImpact.
==========

to

==========
Serialize ResolutionImpact instead of WorldImpact.

R=sigmund@google.com

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

[Johnni Winther, 2016-03-31 07:55:14]

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