Changeset 67066 in webkit for trunk/JavaScriptCore/parser

Timestamp:

Sep 8, 2010, 11:53:33 PM (15 years ago)

Author:

[email protected]

Message:

Removing doneSemicolon label in the lexer
​https://bugs.webkit.org/show_bug.cgi?id=45289

Reviewed by Darin Adler.

As a side effect of moving the multiline comment parsing
to a separate function, an opportunity raised to simplify
the single line comment parsing, and removing doneSemicolon
label. Slight performance increase on --parse-only
tests (from 32.8ms to 31.5ms)

• parser/Lexer.cpp:

(JSC::Lexer::lex):

File:

• trunk/JavaScriptCore/parser/Lexer.cpp (modified) (2 diffs)

trunk/JavaScriptCore/parser/Lexer.cpp

@@ -1026 +1026 @@
         if (lastTokenWasRestrKeyword()) {
             token = SEMICOLON;
-            goto doneSemicolon;
+            m_delimited = true;
+            goto returnToken;
         }
         goto start;
@@ -1048 +1049 @@
     m_atLineStart = true;
     m_terminator = true;
-    if (lastTokenWasRestrKeyword())
-        goto doneSemicolon;
-    goto start;
-
-doneSemicolon:
+    if (!lastTokenWasRestrKeyword())
+        goto start;
+
     token = SEMICOLON;
     m_delimited = true;
     // Fall through into returnToken.
 
-returnToken: {
-    int lineNumber = m_lineNumber;
-    llocp->line = lineNumber;
+returnToken:
+    llocp->line = m_lineNumber;
     llocp->startOffset = startOffset;
     llocp->endOffset = currentOffset();
     m_lastToken = token;
     return token;
-}
 
 returnError: