Changeset 172717 in webkit for trunk/Source/JavaScriptCore/parser/Nodes.h

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/Nodes.h (modified) (1 diff)

trunk/Source/JavaScriptCore/parser/Nodes.h

@@ -1213 +1213 @@
     };
 
+    class EmptyVarExpression : public ExpressionNode {
+    public:
+        EmptyVarExpression(const JSTokenLocation&, const Identifier&);
+
+    private:
+        virtual RegisterID* emitBytecode(BytecodeGenerator&, RegisterID* = 0) override;
+
+        const Identifier& m_ident;
+    };
+
+
     class IfElseNode : public StatementNode {
     public: