Add benchmark output to compiler stats.

Add benchmark output to compiler stats.
Add two compiler stats benchmarks to unit tests

Sample output:
xcodebuild/ReleaseIA32/run_vm_tests  CorelibCompilerStats
NumberOfTokens: 271148
NumClassesParsed: 730
NumFunctionsCompiled: 8987
NumFunctionsOptimized: 0
NumFunctionsParsed: 9694
NumTokensScanned: 0 tokens
ScannerTime: 0 ms
ScannerSpeed: 0 tokens/ms
NumTokensParsed: 531066 tokens
ParserTime: 262 ms
ParserSpeed: 2026 tokens/ms
NumTokensCompiled: 433816 tokens
CompilerTime: 648 ms
CompilerSpeed: 668 tokens/ms
CodeSize: 4428 KB
CodeDensity: 138 tokens/KB
InstrSize: 3127 KB
CorelibCompilerStats(RunTime): 694788

BUG=
R=asiva@google.com

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

[hausner, 2015-09-16 17:01:29]

hausner@google.com changed reviewers:
+ iposva@google.com

[siva, 2015-09-16 17:25:00]

asiva@google.com changed reviewers:
+ asiva@google.com

[siva, 2015-09-16 17:25:00]

lgtm

https://codereview.chromium.org/1352483002/diff/20001/runtime/vm/benchmark_te...
File runtime/vm/benchmark_test.cc (right):

https://codereview.chromium.org/1352483002/diff/20001/runtime/vm/benchmark_te...
runtime/vm/benchmark_test.cc:71: EXPECT(error.IsNull());
The 'EXPECT(error.IsNull())' line seems to make the check above
if (!error.IsNull()) {
  ...
}
redundant.

[hausner, 2015-09-16 18:18:43]

Committed patchset #5 (id:80001) manually as
bb097ec37f4d034622c8ba20cc97931fe1870d0e (presubmit successful).

[hausner, 2015-09-16 18:21:01]

Thank you.

https://codereview.chromium.org/1352483002/diff/20001/runtime/vm/benchmark_te...
File runtime/vm/benchmark_test.cc (right):

https://codereview.chromium.org/1352483002/diff/20001/runtime/vm/benchmark_te...
runtime/vm/benchmark_test.cc:71: EXPECT(error.IsNull());
On 2015/09/16 17:25:00, siva wrote:
> The 'EXPECT(error.IsNull())' line seems to make the check above
> if (!error.IsNull()) {
>   ...
> }
> redundant.

Done, also in the the code above.

[rmacnak, 2015-09-16 19:16:38]

On 2015/09/16 18:21:01, hausner wrote:
> Thank you.
> 
>
https://codereview.chromium.org/1352483002/diff/20001/runtime/vm/benchmark_te...
> File runtime/vm/benchmark_test.cc (right):
> 
>
https://codereview.chromium.org/1352483002/diff/20001/runtime/vm/benchmark_te...
> runtime/vm/benchmark_test.cc:71: EXPECT(error.IsNull());
> On 2015/09/16 17:25:00, siva wrote:
> > The 'EXPECT(error.IsNull())' line seems to make the check above
> > if (!error.IsNull()) {
> >   ...
> > }
> > redundant.
> 
> Done, also in the the code above.

Divide-by-zero:

Program received signal SIGFPE, Arithmetic exception.
0x0000000000895060 in dart::CompilerStats::PrintToZone (this=0x3385e90) at
runtime/vm/compiler_stats.cc:275
275	            (1000 * num_func_tokens_compiled) / compile_usecs);