[WSL] Small cleanup in Evaluator

https://bugs.webkit.org/show_bug.cgi?id=176971

Reviewed by Filip Pizlo.

Use the symbols for break and continue instead of the objects themselves.

No tests because there is no behavior change.

* WebGPUShadingLanguageRI/Evaluator.js:
(Evaluator.prototype.visitWhileLoop):
(Evaluator.prototype.visitDoWhileLoop):
(Evaluator.prototype.visitForLoop):
(Evaluator.prototype.visitBreak):
(Evaluator.prototype.visitContinue):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@222120 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Author: [email protected]

Tools/ChangeLog

@@ -1,3 +1,21 @@
+2017-09-15  Myles C. Maxfield  <[email protected]>
+
+        [WSL] Small cleanup in Evaluator
+        https://bugs.webkit.org/show_bug.cgi?id=176971
+
+        Reviewed by Filip Pizlo.
+
+        Use the symbols for break and continue instead of the objects themselves.
+
+        No tests because there is no behavior change.
+
+        * WebGPUShadingLanguageRI/Evaluator.js:
+        (Evaluator.prototype.visitWhileLoop):
+        (Evaluator.prototype.visitDoWhileLoop):
+        (Evaluator.prototype.visitForLoop):
+        (Evaluator.prototype.visitBreak):
+        (Evaluator.prototype.visitContinue):
+
 2017-09-15  Filip Pizlo  <[email protected]>
 
         WSL Evaluator should only allocate EBuffers when dealing with intrinsics

Tools/WebGPUShadingLanguageRI/Evaluator.js

@@ -180,9 +180,9 @@ class Evaluator extends Visitor {
             try {
                 node.body.visit(this);
             } catch (e) {
-                if (e instanceof Break)
+                if (e == BreakException)
                     break;
-                if (e instanceof Continue)
+                if (e == ContinueException)
                     continue;
                 throw e;
             }
@@ -195,9 +195,9 @@ class Evaluator extends Visitor {
             try {
                 node.body.visit(this);
             } catch (e) {
-                if (e instanceof Break)
+                if (e == BreakException)
                     break;
-                if (e instanceof Continue)
+                if (e == ContinueException)
                     continue;
                 throw e;
             }
@@ -212,9 +212,9 @@ class Evaluator extends Visitor {
             try {
                 node.body.visit(this);
             } catch (e) {
-                if (e instanceof Break)
+                if (e == BreakException)
                     break;
-                if (e instanceof Continue)
+                if (e == ContinueException)
                     continue;
                 throw e;
             }
@@ -223,12 +223,12 @@ class Evaluator extends Visitor {
 
     visitBreak(node)
     {
-        throw node;
+        throw BreakException;
     }
 
     visitContinue(node)
     {
-        throw node;
+        throw ContinueException;
     }
 
     visitLetExpression(node)