Treat yield expressions as an AssignmentPattern error

test/cctest/test-parsing.cc

Index: test/cctest/test-parsing.cc
diff --git a/test/cctest/test-parsing.cc b/test/cctest/test-parsing.cc
index 03da6fd12415964e9242bca45978f73c7437fdd5..76bf274a05e009cf4ed152a660792a3ea9896bd6 100644
--- a/test/cctest/test-parsing.cc
+++ b/test/cctest/test-parsing.cc
@@ -2368,6 +2368,7 @@ TEST(NoErrorsGenerator) {
 
 
 TEST(ErrorsYieldGenerator) {
+  // clang-format off
   const char* context_data[][2] = {
     { "function * gen() {", "}" },
     { "\"use strict\"; function * gen() {", "}" },
@@ -2408,8 +2409,19 @@ TEST(ErrorsYieldGenerator) {
     "yield\n{yield: 42}",
     "yield /* comment */\n {yield: 42}",
     "yield //comment\n {yield: 42}",
+    // Destructuring binding and assignment are both disallowed
+    "var [yield] = [42];",
+    "var {foo: yield} = {a: 42};",
+    "[yield] = [42];",
+    "({a: yield} = {a: 42});",
+    // Also disallow full yield expressions on LHS
+    "var [yield 24] = [42];",
+    "var {foo: yield 24} = {a: 42};",
+    "[yield 24] = [42];",
+    "({a: yield 24} = {a: 42});",

[rossberg, 2016/01/26 19:33:09]

Maybe add cases for {[yield]: x} = {...}, just to be sure.

[adamk, 2016/01/26 19:39:57]

Do you mean for valid parsing of {[yield]: x} (that's a computed property name
as part of a binding pattern)? Or did you mean {x: [yield]}?

[rossberg, 2016/01/26 19:56:26]

The former.

     NULL
   };
+  // clang-format on
 
   RunParserSyncTest(context_data, statement_data, kError);
 }