Simplify AST ID generation.

src/ast.h

 // 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.
 
 #ifndef V8_AST_H_
 #define V8_AST_H_
 
 #include "src/v8.h"
 
 #include "src/assembler.h"
@@ skipping 166 matching lines @@
   int node_count_;
   int feedback_slots_;
 };
 
 
 class AstNode: public ZoneObject {
  public:
   // For generating IDs for AstNodes.
   class IdGen {
    public:
-    explicit IdGen(int id = 0) : id_(id) {}
+    IdGen() : id_(BailoutId::FirstUsable().ToInt()) {}

[wingo, 2014/10/08 13:34:46]

What is the plan with bailout ids -- still a per-function locally unique
incrementing integer?  For parameter destructuring of arrow functions I will
have to transplant expressions from one function to another.  I'm finally
starting on that now:
https://docs.google.com/document/d/1iJWvuakeKsPJOeA-RJ-4bM3Jha5iewQjxKsylgzHE....
 The plan was to see if we could add a post-pass to assign bailout ids, to see
how that would impact perf, because when transplanting you have to preserve
local uniqueness; but is that still a valid strategy?

 
     int GetNextId() { return ReserveIdRange(1); }
     int ReserveIdRange(int n) {
       int tmp = id_;
       id_ += n;
       return tmp;
     }
 
    private:
     int id_;
+
+    DISALLOW_COPY_AND_ASSIGN(IdGen);
   };
 
 #define DECLARE_TYPE_ENUM(type) k##type,
   enum NodeType {
     AST_NODE_LIST(DECLARE_TYPE_ENUM)
     kInvalid = -1
   };
 #undef DECLARE_TYPE_ENUM
 
   void* operator new(size_t size, Zone* zone) {
@@ skipping 3373 matching lines @@
   Zone* zone_;
   Visitor visitor_;
   AstValueFactory* ast_value_factory_;
   AstNode::IdGen* id_gen_;
 };
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_AST_H_