Eagerly disable optimization of %_GeneratorThrow, %_GeneratorNext

src/ast-numbering.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/v8.h"
 
 #include "src/ast.h"
 #include "src/ast-numbering.h"
 #include "src/compiler.h"
 #include "src/scopes.h"
@@ skipping 278 matching lines @@
 }
 
 
 void AstNumberingVisitor::VisitCallRuntime(CallRuntime* node) {
   IncrementNodeCount();
   ReserveFeedbackSlots(node);
   if (node->is_jsruntime()) {
     // Don't try to optimize JS runtime calls because we bailout on them.
     DisableCrankshaft(kCallToAJavaScriptRuntimeFunction);
   }
+  const Runtime::Function* function = node->function();
+  if (function != NULL && function->intrinsic_type == Runtime::INLINE) {

[Michael Starzinger (Google), 2015/01/14 20:42:22]

OMG, pretty please don't. Is this known to fix the problem mentioned in the
issue or is it more a "stab in the dark"?

+    switch (function->function_id) {
+      case Runtime::kInlineGeneratorNext:
+        DisableCrankshaft(kInlinedRuntimeFunctionGeneratorNext);
+        break;
+      case Runtime::kInlineGeneratorThrow:
+        DisableCrankshaft(kInlinedRuntimeFunctionGeneratorThrow);
+        break;
+      default:
+        break;
+    }
+  }
   node->set_base_id(ReserveIdRange(CallRuntime::num_ids()));
   VisitArguments(node->arguments());
 }
 
 
 void AstNumberingVisitor::VisitWithStatement(WithStatement* node) {
   IncrementNodeCount();
   DisableCrankshaft(kWithStatement);
   Visit(node->expression());
   Visit(node->statement());
@@ skipping 260 matching lines @@
   return Finish(node);
 }
 
 
 bool AstNumbering::Renumber(FunctionLiteral* function, Zone* zone) {
   AstNumberingVisitor visitor(zone);
   return visitor.Renumber(function);
 }
 }
 }  // namespace v8::internal