Title: [175938] releases/WebKitGTK/webkit-2.6/Source/WebCore
- Revision
- 175938
- Author
- [email protected]
- Date
- 2014-11-11 09:09:01 -0800 (Tue, 11 Nov 2014)
Log Message
Merge r175462 - Reduce the cost of argumentCount checks in the JS bindings
https://bugs.webkit.org/show_bug.cgi?id=138289
Reviewed by Sam Weinig.
Slightly reduce the cost of argumentCount checks in the JS bindings by
hinting to the compiler that it is UNLIKELY that those methods are
called with too few arguments.
I see a 3% progression on PerformanceTests/DOM/DOMDivWalk.html
performance test.
No new tests, no behavior change.
* bindings/scripts/CodeGeneratorJS.pm:
(GenerateArgumentsCountCheck):
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.6/Source/WebCore/ChangeLog (175937 => 175938)
--- releases/WebKitGTK/webkit-2.6/Source/WebCore/ChangeLog 2014-11-11 16:54:02 UTC (rev 175937)
+++ releases/WebKitGTK/webkit-2.6/Source/WebCore/ChangeLog 2014-11-11 17:09:01 UTC (rev 175938)
@@ -1,3 +1,22 @@
+2014-11-02 Chris Dumez <[email protected]>
+
+ Reduce the cost of argumentCount checks in the JS bindings
+ https://bugs.webkit.org/show_bug.cgi?id=138289
+
+ Reviewed by Sam Weinig.
+
+ Slightly reduce the cost of argumentCount checks in the JS bindings by
+ hinting to the compiler that it is UNLIKELY that those methods are
+ called with too few arguments.
+
+ I see a 3% progression on PerformanceTests/DOM/DOMDivWalk.html
+ performance test.
+
+ No new tests, no behavior change.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateArgumentsCountCheck):
+
2014-10-31 Benjamin Poulain <[email protected]>
Pseudo classes with an escaped parenthesis generate invalid rules
Modified: releases/WebKitGTK/webkit-2.6/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm (175937 => 175938)
--- releases/WebKitGTK/webkit-2.6/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm 2014-11-11 16:54:02 UTC (rev 175937)
+++ releases/WebKitGTK/webkit-2.6/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm 2014-11-11 17:09:01 UTC (rev 175938)
@@ -3169,7 +3169,7 @@
}
if ($numMandatoryParams >= 1)
{
- push(@$outputArray, " if (exec->argumentCount() < $numMandatoryParams)\n");
+ push(@$outputArray, " if (UNLIKELY(exec->argumentCount() < $numMandatoryParams))\n");
push(@$outputArray, " return throwVMError(exec, createNotEnoughArgumentsError(exec));\n");
}
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
