Title: [198803] trunk/Source/_javascript_Core
Revision
198803
Author
[email protected]
Date
2016-03-29 15:11:24 -0700 (Tue, 29 Mar 2016)

Log Message

We don't properly optimize TDZ checks when we declare a let variable without an initializer
https://bugs.webkit.org/show_bug.cgi?id=150453

Reviewed by Mark Lam.

* bytecompiler/NodesCodegen.cpp:
(JSC::EmptyLetExpression::emitBytecode):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (198802 => 198803)


--- trunk/Source/_javascript_Core/ChangeLog	2016-03-29 21:49:42 UTC (rev 198802)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-03-29 22:11:24 UTC (rev 198803)
@@ -1,5 +1,15 @@
 2016-03-29  Saam barati  <[email protected]>
 
+        We don't properly optimize TDZ checks when we declare a let variable without an initializer
+        https://bugs.webkit.org/show_bug.cgi?id=150453
+
+        Reviewed by Mark Lam.
+
+        * bytecompiler/NodesCodegen.cpp:
+        (JSC::EmptyLetExpression::emitBytecode):
+
+2016-03-29  Saam barati  <[email protected]>
+
         Allow builtin JS functions to be intrinsics
         https://bugs.webkit.org/show_bug.cgi?id=155960
 

Modified: trunk/Source/_javascript_Core/bytecompiler/NodesCodegen.cpp (198802 => 198803)


--- trunk/Source/_javascript_Core/bytecompiler/NodesCodegen.cpp	2016-03-29 21:49:42 UTC (rev 198802)
+++ trunk/Source/_javascript_Core/bytecompiler/NodesCodegen.cpp	2016-03-29 22:11:24 UTC (rev 198803)
@@ -2151,6 +2151,8 @@
         generator.emitProfileType(value.get(), var, position(), JSTextPosition(-1, position().offset + m_ident.length(), -1)); 
     }
 
+    generator.liftTDZCheckIfPossible(var);
+
     // It's safe to return null here because this node will always be a child node of DeclarationStatement which ignores our return value.
     return nullptr;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to