Implement optimized mixin application in dart2js

Implement optimized mixin application in dart2js

This prepares for equivalence testing of shared mixins.

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

[Johnni Winther, 2017-07-18 14:45:02]

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

[Siggi Cherem (dart-lang), 2017-07-19 04:38:35]

I didn't get a chance to review the ported algorithm in detail, but I'm OK
submitting this if you want to move forward. If you'd like me to look in more
detail, let me know.

https://codereview.chromium.org/2983013002/diff/1/pkg/compiler/lib/src/resolu...
File pkg/compiler/lib/src/resolution/class_hierarchy.dart (right):

https://codereview.chromium.org/2983013002/diff/1/pkg/compiler/lib/src/resolu...
pkg/compiler/lib/src/resolution/class_hierarchy.dart:338: /// which is crucial
for memory usage and the Dart VM's bootstrap
VM bootstrap? I just realized this code was ported over from fasta :)?

Maybe add a comment about it on the first line of this long comment:
/// The following algorithm is duplicated from fasta/path/to/file.dart
/// This code will be deleted as soon as we stop testing equivalence of the old
and new
/// compiler models.

[Johnni Winther, 2017-07-19 11:13:57]

Description was changed from

==========
Implement optimized mixin application in dart2js

This prepares for equivalence testing of shared mixins.
==========

to

==========
Implement optimized mixin application in dart2js

This prepares for equivalence testing of shared mixins.

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

[Johnni Winther, 2017-07-19 11:13:58]

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

[Johnni Winther, 2017-07-19 11:33:59]

https://codereview.chromium.org/2983013002/diff/1/pkg/compiler/lib/src/resolu...
File pkg/compiler/lib/src/resolution/class_hierarchy.dart (right):

https://codereview.chromium.org/2983013002/diff/1/pkg/compiler/lib/src/resolu...
pkg/compiler/lib/src/resolution/class_hierarchy.dart:338: /// which is crucial
for memory usage and the Dart VM's bootstrap
On 2017/07/19 04:38:34, Siggi Cherem (dart-lang) wrote:
> VM bootstrap? I just realized this code was ported over from fasta :)?
> 
> Maybe add a comment about it on the first line of this long comment:
> /// The following algorithm is duplicated from fasta/path/to/file.dart
> /// This code will be deleted as soon as we stop testing equivalence of the
old
> and new
> /// compiler models.

Done.