Closed Bug 732939 Opened 13 years ago Closed 7 years ago

acceptance/runtests.py should support --showtimes for AOT compile to log build time

Categories

(Tamarin Graveyard :: Virtual Machine, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: dschaffe, Unassigned)

Details

Attachments

(1 file, 3 obsolete files)

In runtestBase.py the --showtimes should output the time to compile each test from adt. We could output a format like: AOT compiling abcasm/bug_558442.abc time 660.0 The --logjunit flag could parse the output to include the aot time.
Attached patch patch (obsolete) — Splinter Review
Attached patch patch (obsolete) — Splinter Review
Attachment #602865 - Attachment is obsolete: true
Attached patch patch (obsolete) — Splinter Review
Attachment #602868 - Attachment is obsolete: true
Comment on attachment 602870 [details] [diff] [review] patch Review of attachment 602870 [details] [diff] [review]: ----------------------------------------------------------------- ::: test/util/runtestBase.py @@ +1213,3 @@ > (f,err,exitcode) = self.compile_aot(testdir+".abc", [self.abcasmShell+'.abc']) > + if self.show_time: > + self.js_print('AOT compiling %s time %.1f' % (testdir+".abc",time.time()-start_time)) Should we be tracking compilation time if compilation failed?
Attached patch patchSplinter Review
Attachment #602870 - Attachment is obsolete: true
Attachment #603686 - Attachment is patch: true
changeset: 7259:782d8f327cca user: Dan Schaffer <dschaffe@adobe.com> summary: Bug 732939: log compilation time for AOT during --rebuildtests (r+brbaker) http://hg.mozilla.org/tamarin-redux/rev/782d8f327cca
changeset: 7259:782d8f327cca user: Dan Schaffer <dschaffe@adobe.com> summary: Bug 732939: log compilation time for AOT during --rebuildtests (r+brbaker) http://hg.mozilla.org/tamarin-redux/rev/782d8f327cca
changeset: 7268:511843343487 user: Dan Schaffer <Dan.Schaffer@adobe.com> summary: bug 732939: fixes for logging AOT compilation to junit xml format http://hg.mozilla.org/tamarin-redux/rev/511843343487
changeset: 7325:a645ccd0a56a user: Dan Schaffer <Dan.Schaffer@adobe.com> summary: bug 732939: fix aot --showtimes parsing bug for junit xml report generation. http://hg.mozilla.org/tamarin-redux/rev/a645ccd0a56a
changeset: 7331:2e66898b6637 user: Dan Schaffer <Dan.Schaffer@adobe.com> summary: bug 732939: fixes for junit reporting xml http://hg.mozilla.org/tamarin-redux/rev/2e66898b6637
Tamarin is a dead project now. Mass WONTFIX.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: