Tweak the TurboFan pipeline for stub compilation.

src/compiler/pipeline.cc

 // Copyright 2014 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/compiler/pipeline.h"
 
 #include <fstream>  // NOLINT(readability/streams)
 #include <sstream>
 
 #include "src/base/platform/elapsed-timer.h"
@@ skipping 295 matching lines @@
 
 
 class AstGraphBuilderWithPositions : public AstGraphBuilder {
  public:
   AstGraphBuilderWithPositions(Zone* local_zone, CompilationInfo* info,
                                JSGraph* jsgraph,
                                LoopAssignmentAnalysis* loop_assignment,
                                SourcePositionTable* source_positions)
       : AstGraphBuilder(local_zone, info, jsgraph, loop_assignment),
         source_positions_(source_positions),
-        start_position_(info->shared_info()->start_position()) {}
+        start_position_(info->shared_info()->start_position()),
+        stack_check_(!info->IsStub()) {}
 
   bool CreateGraph(bool constant_context) {

[Michael Starzinger, 2015/03/17 17:28:55]

nit: Can we add a "bool stack_check" to the CreateGraph method here and move the
"info->IsStub()" to the call-site? That would make it less hidden away.

[Sven Panne, 2015/03/18 08:45:55]

Done.

     SourcePositionTable::Scope pos_scope(source_positions_, start_position_);
-    return AstGraphBuilder::CreateGraph(constant_context);
+    return AstGraphBuilder::CreateGraph(constant_context, stack_check_);
   }
 
 #define DEF_VISIT(type)                                               \
   void Visit##type(type* node) OVERRIDE {                             \
     SourcePositionTable::Scope pos(source_positions_,                 \
                                    SourcePosition(node->position())); \
     AstGraphBuilder::Visit##type(node);                               \
   }
   AST_NODE_LIST(DEF_VISIT)
 #undef DEF_VISIT
 
  private:
   SourcePositionTable* source_positions_;
   SourcePosition start_position_;
+  bool stack_check_;

[Michael Starzinger, 2015/03/17 17:28:55]

nit: This field is not required once the above comment is addressed.

[Sven Panne, 2015/03/18 08:45:55]

Done.

 };
 
 
 namespace {
 
 class SourcePositionWrapper : public Reducer {
  public:
   SourcePositionWrapper(Reducer* reducer, SourcePositionTable* table)
       : reducer_(reducer), table_(table) {}
   virtual ~SourcePositionWrapper() {}
@@ skipping 850 matching lines @@
 
   if (FLAG_trace_turbo && !data->MayHaveUnverifiableGraph()) {
     TurboCfgFile tcf(data->isolate());
     tcf << AsC1VAllocator("CodeGen", data->register_allocator());
   }
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8