Bug 210938 - Support `--report-execution-time` to report execution time for each JSC stress test
Summary: Support `--report-execution-time` to report execution time for each JSC stres...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Yusuke Suzuki
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-04-23 15:24 PDT by Yusuke Suzuki
Modified: 2020-04-23 17:13 PDT (History)
2 users (show)

See Also:


Attachments
Patch (5.98 KB, patch)
2020-04-23 15:26 PDT, Yusuke Suzuki
no flags Details | Formatted Diff | Diff
Patch (6.15 KB, patch)
2020-04-23 15:37 PDT, Yusuke Suzuki
saam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yusuke Suzuki 2020-04-23 15:24:53 PDT
Support `--report-execution-time` to report execution time for each JSC stress test
Comment 1 Yusuke Suzuki 2020-04-23 15:26:04 PDT
Created attachment 397391 [details]
Patch
Comment 2 Yusuke Suzuki 2020-04-23 15:37:36 PDT
Created attachment 397393 [details]
Patch
Comment 3 Yusuke Suzuki 2020-04-23 17:06:42 PDT
Comment on attachment 397393 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=397393&action=review

> Tools/Scripts/webkitruby/jsc-stress-test-writer-default.rb:284
> +            outp.puts "START_TIME=$SECONDS"

I'll also remove this when $reportExecutionTime is false anyway.
Comment 4 Yusuke Suzuki 2020-04-23 17:08:41 PDT
Ensured it works locally.
Comment 5 Yusuke Suzuki 2020-04-23 17:12:06 PDT
Committed r260610: <https://trac.webkit.org/changeset/260610>
Comment 6 Radar WebKit Bug Importer 2020-04-23 17:13:19 PDT
<rdar://problem/62271127>