Title: [222618] trunk/Tools
- Revision
- 222618
- Author
- [email protected]
- Date
- 2017-09-28 11:25:28 -0700 (Thu, 28 Sep 2017)
Log Message
[Re-landing] Turn on exception scope verification for JSC tests.
https://bugs.webkit.org/show_bug.cgi?id=162351
<rdar://problem/29563911>
Reviewed by Saam Barati.
Update: I'm re-landing this patch now that test262 exception check validation
failures have been fixed in r222617. testapi still has some validation failures,
but this patch does not affect testapi.
Added the option to --validateExceptionChecks=true option to BASE_OPTIONS in
run-jsc-stress-tests. This turns on exception scope verification on JSC test
runs (which currently does not include testapi).
Some stats on time to run JSC stress and mozilla tests:
1. Release build w/o --validateExceptionChecks=true: real 16m22.544s, user 156m24.080s, sys 123m3.649s
2. Debug build w/o --validateExceptionChecks=true: real 78m34.206s, user 1661m57.008s, sys 73m21.177s
3. Debug build w/ --validateExceptionChecks=true: real 77m41.106s, user 1656m13.924s, sys 73m42.309s
4. Debug build w/ --validateExceptionChecks=true --dumpSimulatedThrows=true: real 92m56.918s, user 2012m56.441s, sys 75m14.174s
The stats shows that (2) and (3) has effectively no time difference. Hence, the
cost of enabling --validateExceptionChecks=true is not significant.
It would be nice to enable --dumpSimulatedThrows=true as well, but (4) is about
21% slower than (3). To avoid making debug test runs a lot slower, we'll leave
--dumpSimulatedThrows=true off. We can manually add that when we see a regression
and need to debug the issue. Otherwise, we wont pay the price for it.
* Scripts/run-jsc-stress-tests:
Modified Paths
Diff
Modified: trunk/Tools/ChangeLog (222617 => 222618)
--- trunk/Tools/ChangeLog 2017-09-28 18:09:09 UTC (rev 222617)
+++ trunk/Tools/ChangeLog 2017-09-28 18:25:28 UTC (rev 222618)
@@ -1,3 +1,35 @@
+2017-09-28 Mark Lam <[email protected]>
+
+ [Re-landing] Turn on exception scope verification for JSC tests.
+ https://bugs.webkit.org/show_bug.cgi?id=162351
+ <rdar://problem/29563911>
+
+ Reviewed by Saam Barati.
+
+ Update: I'm re-landing this patch now that test262 exception check validation
+ failures have been fixed in r222617. testapi still has some validation failures,
+ but this patch does not affect testapi.
+
+ Added the option to --validateExceptionChecks=true option to BASE_OPTIONS in
+ run-jsc-stress-tests. This turns on exception scope verification on JSC test
+ runs (which currently does not include testapi).
+
+ Some stats on time to run JSC stress and mozilla tests:
+ 1. Release build w/o --validateExceptionChecks=true: real 16m22.544s, user 156m24.080s, sys 123m3.649s
+ 2. Debug build w/o --validateExceptionChecks=true: real 78m34.206s, user 1661m57.008s, sys 73m21.177s
+ 3. Debug build w/ --validateExceptionChecks=true: real 77m41.106s, user 1656m13.924s, sys 73m42.309s
+ 4. Debug build w/ --validateExceptionChecks=true --dumpSimulatedThrows=true: real 92m56.918s, user 2012m56.441s, sys 75m14.174s
+
+ The stats shows that (2) and (3) has effectively no time difference. Hence, the
+ cost of enabling --validateExceptionChecks=true is not significant.
+
+ It would be nice to enable --dumpSimulatedThrows=true as well, but (4) is about
+ 21% slower than (3). To avoid making debug test runs a lot slower, we'll leave
+ --dumpSimulatedThrows=true off. We can manually add that when we see a regression
+ and need to debug the issue. Otherwise, we wont pay the price for it.
+
+ * Scripts/run-jsc-stress-tests:
+
2017-09-27 Alex Christensen <[email protected]>
Add WKContentRuleList notify action type
Modified: trunk/Tools/Scripts/run-jsc-stress-tests (222617 => 222618)
--- trunk/Tools/Scripts/run-jsc-stress-tests 2017-09-28 18:09:09 UTC (rev 222617)
+++ trunk/Tools/Scripts/run-jsc-stress-tests 2017-09-28 18:25:28 UTC (rev 222618)
@@ -451,7 +451,7 @@
$numPasses = 0
# We force all tests to use a smaller (1.5M) stack so that stack overflow tests can run faster.
-BASE_OPTIONS = ["--useFTLJIT=false", "--useFunctionDotArguments=true", "--maxPerThreadStackUsage=1572864"]
+BASE_OPTIONS = ["--useFTLJIT=false", "--useFunctionDotArguments=true", "--validateExceptionChecks=true", "--maxPerThreadStackUsage=1572864"]
EAGER_OPTIONS = ["--thresholdForJITAfterWarmUp=10", "--thresholdForJITSoon=10", "--thresholdForOptimizeAfterWarmUp=20", "--thresholdForOptimizeAfterLongWarmUp=20", "--thresholdForOptimizeSoon=20", "--thresholdForFTLOptimizeAfterWarmUp=20", "--thresholdForFTLOptimizeSoon=20", "--maximumEvalCacheableSourceLength=150000", "--useEagerCodeBlockJettisonTiming=true"]
# NOTE: Tests rely on this using scribbleFreeCells.
NO_CJIT_OPTIONS = ["--useConcurrentJIT=false", "--thresholdForJITAfterWarmUp=100", "--scribbleFreeCells=true"]
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
