RESOLVED FIXED 160021
Clarify testing mode names in run-jsc-stress-tests
https://bugs.webkit.org/show_bug.cgi?id=160021
Summary Clarify testing mode names in run-jsc-stress-tests
Csaba Osztrogonác
Reported 2016-07-21 06:12:34 PDT
https://trac.webkit.org/changeset/203440 removed the possibility to skip redundant tests on non FTL platforms which made testing terrible slow ( just to be able to paint some bots green ... ) r203440 stated that default should really mean default, not default with disabled FTL JIT. Let's do the following renaming to have clear names: - runMozillaTestDefault -> runMozillaTestNoFTL - runMozillaTestDefaultFTL -> runMozillaTestDefault - runDefault -> runNoFTL - runDefaultFTL -> runDefault
Attachments
Patch (37.88 KB, patch)
2016-07-21 06:20 PDT, Csaba Osztrogonác
no flags
Patch (40.05 KB, patch)
2016-07-21 08:07 PDT, Csaba Osztrogonác
no flags
Patch (40.93 KB, patch)
2016-07-21 10:28 PDT, Csaba Osztrogonác
no flags
Csaba Osztrogonác
Comment 1 2016-07-21 06:20:15 PDT
Created attachment 284210 [details] Patch no behavior change, only renaming
Csaba Osztrogonác
Comment 2 2016-07-21 07:32:23 PDT
(In reply to comment #0) > Let's do the following renaming to have clear names: > - runMozillaTestDefault -> runMozillaTestNoFTL > - runMozillaTestDefaultFTL -> runMozillaTestDefault > - runDefault -> runNoFTL > - runDefaultFTL -> runDefault and - runLayoutTestDefault -> runLayoutTestNoFTL - runLayoutTestDefaultFTL -> runLayoutTestDefault
Csaba Osztrogonác
Comment 3 2016-07-21 08:07:19 PDT
Csaba Osztrogonác
Comment 4 2016-07-21 10:26:02 PDT
and - runNoisyTestDefault to runNoisyTestNoFTL - runNoisyTestDefaultFTL to runNoisyTestDefault
Csaba Osztrogonác
Comment 5 2016-07-21 10:28:47 PDT
Mark Lam
Comment 6 2016-07-21 10:58:42 PDT
Comment on attachment 284225 [details] Patch r=me if EWS bots are happy.
Csaba Osztrogonác
Comment 7 2016-07-21 11:09:44 PDT
Comment on attachment 284225 [details] Patch Clearing flags on attachment: 284225 Committed r203512: <http://trac.webkit.org/changeset/203512>
Csaba Osztrogonác
Comment 8 2016-07-21 11:09:53 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.