Changeset 172717 in webkit for trunk/Source/JavaScriptCore/parser/Parser.cpp

Timestamp:

Aug 18, 2014, 12:11:41 PM (11 years ago)

Author:

[email protected]

Message:

The parser should generate AST nodes the var declarations with no initializers
​https://bugs.webkit.org/show_bug.cgi?id=135545

Patch by Saam Barati <​[email protected]> on 2014-08-18
Reviewed by Geoffrey Garen.

Currently, JSC's parser ignores variable declarations
that have no assignment initializer value because all
variables are implicitly assigned to undefined. But,
type profiling needs an AST node to be generated for these
empty variable declarations because it needs to be able to
profile their text locations and to see that their type
is undefined.

• bytecompiler/NodesCodegen.cpp:

(JSC::EmptyVarExpression::emitBytecode):

• parser/ASTBuilder.h:

(JSC::ASTBuilder::createVarStatement):
(JSC::ASTBuilder::createEmptyVarExpression):

• parser/NodeConstructors.h:

(JSC::EmptyVarExpression::EmptyVarExpression):

• parser/Nodes.h:
• parser/Parser.cpp:

(JSC::Parser<LexerType>::parseVarDeclarationList):

• parser/SyntaxChecker.h:

(JSC::SyntaxChecker::createEmptyVarExpression):

File:

• trunk/Source/JavaScriptCore/parser/Parser.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/parser/Parser.cpp

@@ -459 +459 @@
         if (match(IDENT)) {
             JSTextPosition varStart = tokenStartPosition();
+            JSTokenLocation varStartLocation(tokenLocation());
             identStart = varStart;
             const Identifier* name = m_token.m_data.ident;
@@ -476 +477 @@
                 
                 node = context.createAssignResolve(location, *name, initializer, varStart, varDivot, lastTokenEndPosition());
-            }
+            } else
+                node = context.createEmptyVarExpression(varStartLocation, *name);
         } else {
             lastIdent = 0;
@@ -491 +493 @@
         }
         
-        if (hasInitializer) {
-            if (!varDecls)
-                varDecls = node;
-            else
-                varDecls = context.combineCommaNodes(location, varDecls, node);
-        }
+        if (!varDecls)
+            varDecls = node;
+        else
+            varDecls = context.combineCommaNodes(location, varDecls, node);
     } while (match(COMMA));
     if (lastIdent)