Title: [171852] trunk/Source/_javascript_Core
- Revision
- 171852
- Author
- [email protected]
- Date
- 2014-07-31 08:21:42 -0700 (Thu, 31 Jul 2014)
Log Message
NewFunctionExpression and NewFunctionNoCheck should setHaveStructures(true)
https://bugs.webkit.org/show_bug.cgi?id=135430
Reviewed by Mark Hahnenberg.
We already handled this correctly after the ftlopt merge, but it's useful to have the test.
* tests/stress/new-function-_expression_-has-structures.js: Added.
(foo.f):
(foo.f.prototype.f):
(foo):
Modified Paths
Added Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (171851 => 171852)
--- trunk/Source/_javascript_Core/ChangeLog 2014-07-31 15:08:29 UTC (rev 171851)
+++ trunk/Source/_javascript_Core/ChangeLog 2014-07-31 15:21:42 UTC (rev 171852)
@@ -1,3 +1,17 @@
+2014-07-30 Filip Pizlo <[email protected]>
+
+ NewFunctionExpression and NewFunctionNoCheck should setHaveStructures(true)
+ https://bugs.webkit.org/show_bug.cgi?id=135430
+
+ Reviewed by Mark Hahnenberg.
+
+ We already handled this correctly after the ftlopt merge, but it's useful to have the test.
+
+ * tests/stress/new-function-_expression_-has-structures.js: Added.
+ (foo.f):
+ (foo.f.prototype.f):
+ (foo):
+
2014-07-30 Andreas Kling <[email protected]>
Speculative Windows build fix.
Added: trunk/Source/_javascript_Core/tests/stress/new-function-_expression_-has-structures.js (0 => 171852)
--- trunk/Source/_javascript_Core/tests/stress/new-function-_expression_-has-structures.js (rev 0)
+++ trunk/Source/_javascript_Core/tests/stress/new-function-_expression_-has-structures.js 2014-07-31 15:21:42 UTC (rev 171852)
@@ -0,0 +1,11 @@
+function foo() {
+ var f = function() { return 42 };
+ f.prototype.f = function() { return 43; };
+ return f.prototype.f;
+}
+
+noInline(foo);
+
+for (var i = 0; i < 10000; ++i)
+ foo();
+
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
