Add an R script that does some statistical tests on benchmark results

[benchmarks] Add R script for statistical tests on benchmark results

NOTRY=true

Committed: https://crrev.com/4094d6b567de3cc316821a6374445a7f51f09635
Cr-Commit-Position: refs/heads/master@{#33769}

[sigurds, 2016-02-02 13:55:21]

Description was changed from

==========
Add an R script that does some statistical tests on benchmark results

BUG=
==========

to

==========
[benchmarks] Add R script for statistical tests on benchmark results


NOTRY=true
==========

[sigurds, 2016-02-02 13:55:21]

sigurds@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[sigurds, 2016-02-02 14:14:29]

This is the R script I used to evaluate escape analysis. Maybe you want it in
the repo. In any case, this CL has a copy now.

[Michael Achenbach, 2016-02-02 15:53:09]

machenbach@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2016-02-02 15:53:10]

Maybe we should start adding more subdirs to tools. Tools itself is quite
bloated. E.g. "perf"?

https://codereview.chromium.org/1659043002/diff/40001/tools/statistics-for-js...
File tools/statistics-for-json.R (right):

https://codereview.chromium.org/1659043002/diff/40001/tools/statistics-for-js...
tools/statistics-for-json.R:1: # Copyright 2016 the V8 project authors. All
rights reserved.
Could you add a new, short copyright header? E.g. like:
https://code.google.com/p/chromium/codesearch#chromium/src/v8/tools/run_perf....

[sigurds, 2016-02-02 16:02:02]

Move it to tools/perf/ and added the short copyright header.

https://codereview.chromium.org/1659043002/diff/40001/tools/statistics-for-js...
File tools/statistics-for-json.R (right):

https://codereview.chromium.org/1659043002/diff/40001/tools/statistics-for-js...
tools/statistics-for-json.R:1: # Copyright 2016 the V8 project authors. All
rights reserved.
On 2016/02/02 15:53:10, Michael Achenbach wrote:
> Could you add a new, short copyright header? E.g. like:
>
https://code.google.com/p/chromium/codesearch#chromium/src/v8/tools/run_perf....

Done.

[Michael Achenbach, 2016-02-05 09:37:17]

lgtm (rubber-stamp). I didn't really review the code - no R-foo. But I'd be fine
with keeping this as experimental stuff. Could you maybe add some short
instructions on how to use this? E.g. set of cmd-line calls, as long as the data
is public.

[sigurds, 2016-02-05 12:23:49]

The CQ bit was checked by sigurds@chromium.org

[sigurds, 2016-02-05 12:23:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
machenbach@chromium.org
Link to the patchset: https://codereview.chromium.org/1659043002/#ps80001
(title: "Add example command line")

[commit-bot: I haz the power, 2016-02-05 12:23:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1659043002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1659043002/80001

[commit-bot: I haz the power, 2016-02-05 12:29:39]

Description was changed from

==========
[benchmarks] Add R script for statistical tests on benchmark results


NOTRY=true
==========

to

==========
[benchmarks] Add R script for statistical tests on benchmark results


NOTRY=true
==========

[commit-bot: I haz the power, 2016-02-05 12:29:40]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-02-05 12:30:05]

Description was changed from

==========
[benchmarks] Add R script for statistical tests on benchmark results


NOTRY=true
==========

to

==========
[benchmarks] Add R script for statistical tests on benchmark results

NOTRY=true

Committed: https://crrev.com/4094d6b567de3cc316821a6374445a7f51f09635
Cr-Commit-Position: refs/heads/master@{#33769}
==========

[commit-bot: I haz the power, 2016-02-05 12:30:06]

Patchset 5 (id:??) landed as
https://crrev.com/4094d6b567de3cc316821a6374445a7f51f09635
Cr-Commit-Position: refs/heads/master@{#33769}