Add a perf test for generating linked summaries.

Add a perf test for generating linked summaries.

R=sigmund@google.com

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

[Paul Berry, 2016-12-07 22:10:16]

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

[Siggi Cherem (dart-lang), 2016-12-07 23:21:29]

https://codereview.chromium.org/2556723005/diff/1/pkg/front_end/tool/perf.dart
File pkg/front_end/tool/perf.dart (right):

https://codereview.chromium.org/2556723005/diff/1/pkg/front_end/tool/perf.dar...
pkg/front_end/tool/perf.dart:300: setupForLink(libraryUris, getUnit,
getDeclaredVariable);
is there a difference between prelink and setupForLink?

If not, could we refactor the logic in prelinkedSummarizeFilzes so we can reuse
some code and only do the additional link step here?

[Paul Berry, 2016-12-07 23:54:41]

https://codereview.chromium.org/2556723005/diff/1/pkg/front_end/tool/perf.dart
File pkg/front_end/tool/perf.dart (right):

https://codereview.chromium.org/2556723005/diff/1/pkg/front_end/tool/perf.dar...
pkg/front_end/tool/perf.dart:300: setupForLink(libraryUris, getUnit,
getDeclaredVariable);
On 2016/12/07 23:21:29, Siggi Cherem (dart-lang) wrote:
> is there a difference between prelink and setupForLink?
> 
> If not, could we refactor the logic in prelinkedSummarizeFilzes so we can
reuse
> some code and only do the additional link step here?

Good point.  PTAL.

[Siggi Cherem (dart-lang), 2016-12-08 00:24:38]

lgtm

[Paul Berry, 2016-12-08 00:27:32]

Description was changed from

==========
Add a perf test for generating linked summaries.
==========

to

==========
Add a perf test for generating linked summaries.

R=sigmund@google.com

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

[Paul Berry, 2016-12-08 00:27:33]

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