Improve measuring

Improve measuring

Committed: https://code.google.com/p/dart/source/detail?r=15579

[ngeoffray, 2012-11-27 09:14:07]

LGTM

https://codereview.chromium.org/11421060/diff/1/dart/sdk/lib/_internal/compil...
File dart/sdk/lib/_internal/compiler/implementation/compiler.dart (right):

https://codereview.chromium.org/11421060/diff/1/dart/sdk/lib/_internal/compil...
dart/sdk/lib/_internal/compiler/implementation/compiler.dart:360:
unreported.measure(() => runCompiler(uri));
Will that just measure the non-measured time at the beginning of runCompiler and
the non-measured time at the end of runCompiler? 'unreported' makes you think it
also contains all non measured times, but that's not it right?

[ahe, 2012-11-30 12:00:37]

https://codereview.chromium.org/11421060/diff/1/dart/sdk/lib/_internal/compil...
File dart/sdk/lib/_internal/compiler/implementation/compiler.dart (right):

https://codereview.chromium.org/11421060/diff/1/dart/sdk/lib/_internal/compil...
dart/sdk/lib/_internal/compiler/implementation/compiler.dart:360:
unreported.measure(() => runCompiler(uri));
On 2012/11/27 09:14:07, ngeoffray wrote:
> Will that just measure the non-measured time at the beginning of runCompiler
and
> the non-measured time at the end of runCompiler? 'unreported' makes you think
it
> also contains all non measured times, but that's not it right?

I have changed it.  Now it looks like this:

info: Reading input files took 95msec
info: LibraryLoader took 40msec
info: Scanner took 174msec
info: Diet Parser took 138msec
info: Parser took 379msec
info: Patching Parser took 9msec
info: Resolver took 908msec
info: Closure Simplifier took 108msec
info: Type checker took 142msec
info: Type inference took 45msec
info: ConstantHandler took 35msec
info: Enqueue took 484msec
info: SSA builder took 759msec
info: SSA optimizer took 1445msec
info: SSA code generator took 2176msec
info: CodeEmitter took 386msec
info: Total compile-time 7518msec; unaccounted 195msec

[ngeoffray, 2012-11-30 12:05:28]

LGTM