Incorporate some review comments.

runtime/vm/intermediate_language.h

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef VM_INTERMEDIATE_LANGUAGE_H_
 #define VM_INTERMEDIATE_LANGUAGE_H_
 
 #include "vm/allocation.h"
 #include "vm/growable_array.h"
 #include "vm/handles_impl.h"
@@ skipping 194 matching lines @@
   void flip_mark() { mark_ = !mark_; }
 
 #define INSTRUCTION_TYPE_CHECK(type)                                           \
   virtual bool Is##type() const { return false; }                              \
   virtual type##Instr* As##type() { return NULL; }
 FOR_EACH_INSTRUCTION(INSTRUCTION_TYPE_CHECK)
 #undef INSTRUCTION_TYPE_CHECK
 
  private:
   bool mark_;
+
+  DISALLOW_COPY_AND_ASSIGN(Instruction);
 };
 
 
 // Basic block entries are administrative nodes.  Joins are the only nodes
 // with multiple predecessors.  Targets are the other basic block entries.
 // The types enforce edge-split form---joins are forbidden as the successors
 // of branches.
 class BlockEntryInstr : public Instruction {
  public:
   virtual bool IsBlockEntry() const { return true; }
 
   static BlockEntryInstr* cast(Instruction* instr) {
     ASSERT(instr->IsBlockEntry());
     return reinterpret_cast<BlockEntryInstr*>(instr);
   }
 
   intptr_t block_number() const { return block_number_; }
   void set_block_number(intptr_t number) { block_number_ = number; }
 
  protected:
   BlockEntryInstr() : Instruction(), block_number_(-1) { }
 
  private:
   intptr_t block_number_;
+
+  DISALLOW_COPY_AND_ASSIGN(BlockEntryInstr);
 };
 
 
 class JoinEntryInstr : public BlockEntryInstr {
  public:
   JoinEntryInstr() : BlockEntryInstr(), successor_(NULL) { }
 
   DECLARE_INSTRUCTION(JoinEntry)
 
   virtual void SetSuccessor(Instruction* instr) {
     ASSERT(successor_ == NULL);
     successor_ = instr;
   }
 
   virtual void Postorder(GrowableArray<BlockEntryInstr*>* block_entries);
 
  private:
   Instruction* successor_;
+
+  DISALLOW_COPY_AND_ASSIGN(JoinEntryInstr);
 };
 
 
 class TargetEntryInstr : public BlockEntryInstr {
  public:
-  TargetEntryInstr() : BlockEntryInstr(), block_number_(-1), successor_(NULL) {
+  TargetEntryInstr() : BlockEntryInstr(), successor_(NULL) {
   }
 
   DECLARE_INSTRUCTION(TargetEntry)
 
   virtual void SetSuccessor(Instruction* instr) {
     ASSERT(successor_ == NULL);
     successor_ = instr;
   }
 
   virtual void Postorder(GrowableArray<BlockEntryInstr*>* block_entries);
 
  private:
-  intptr_t block_number_;
   Instruction* successor_;
+
+  DISALLOW_COPY_AND_ASSIGN(TargetEntryInstr);
 };
 
 
 class DoInstr : public Instruction {
  public:
   explicit DoInstr(Computation* comp)
       : Instruction(), computation_(comp), successor_(NULL) { }
 
   DECLARE_INSTRUCTION(Do)
 
   Computation* computation() const { return computation_; }
 
   virtual void SetSuccessor(Instruction* instr) {
     ASSERT(successor_ == NULL);
     successor_ = instr;
   }
 
   virtual void Postorder(GrowableArray<BlockEntryInstr*>* block_entries);
 
  private:
   Computation* computation_;
   Instruction* successor_;
+
+  DISALLOW_COPY_AND_ASSIGN(DoInstr);
 };
 
 
 class BindInstr : public Instruction {
  public:
   BindInstr(intptr_t temp_index, Computation* computation)
       : Instruction(),
         temp_index_(temp_index),
         computation_(computation),
         successor_(NULL) { }
 
   DECLARE_INSTRUCTION(Bind)
 
   intptr_t temp_index() const { return temp_index_; }
   Computation* computation() const { return computation_; }
 
   virtual void SetSuccessor(Instruction* instr) {
     ASSERT(successor_ == NULL);
     successor_ = instr;
   }
 
   virtual void Postorder(GrowableArray<BlockEntryInstr*>* block_entries);
 
  private:
   const intptr_t temp_index_;
   Computation* computation_;
   Instruction* successor_;
+
+  DISALLOW_COPY_AND_ASSIGN(BindInstr);
 };
 
 
 class ReturnInstr : public Instruction {
  public:
   explicit ReturnInstr(Value* value) : Instruction(), value_(value) { }
 
   DECLARE_INSTRUCTION(Return)
 
   Value* value() const { return value_; }
 
   virtual void SetSuccessor(Instruction* instr) { UNREACHABLE(); }
 
   virtual void Postorder(GrowableArray<BlockEntryInstr*>* block_entries);
 
  private:
   Value* value_;
+
+  DISALLOW_COPY_AND_ASSIGN(ReturnInstr);
 };
 
 
 class BranchInstr : public Instruction {
  public:
   explicit BranchInstr(Value* value)
       : Instruction(),
         value_(value),
         true_successor_(NULL),
         false_successor_(NULL) { }
 
   DECLARE_INSTRUCTION(Branch)
 
   Value* value() const { return value_; }
   TargetEntryInstr* true_successor() const { return true_successor_; }
   TargetEntryInstr* false_successor() const { return false_successor_; }
 
   TargetEntryInstr** true_successor_address() { return &true_successor_; }
   TargetEntryInstr** false_successor_address() { return &false_successor_; }
 
   virtual void SetSuccessor(Instruction* instr) { UNREACHABLE(); }
 
   virtual void Postorder(GrowableArray<BlockEntryInstr*>* block_entries);
 
  private:
   Value* value_;
   TargetEntryInstr* true_successor_;
   TargetEntryInstr* false_successor_;
+
+  DISALLOW_COPY_AND_ASSIGN(BranchInstr);
 };
 
 #undef DECLARE_INSTRUCTION
 
 
 class InstructionVisitor {
  public:
   InstructionVisitor() { }
   virtual ~InstructionVisitor() { }
 
   // Visit each block in the array list in reverse, and for each block its
   // instructions in order from the block entry to exit.
   void VisitBlocks(const GrowableArray<BlockEntryInstr*>& block_order);
 
 #define DECLARE_VISIT(type)                             \
   virtual void Visit##type(type##Instr* instr) { }
   FOR_EACH_INSTRUCTION(DECLARE_VISIT)
 #undef DECLARE_VISIT
 
  private:
   DISALLOW_COPY_AND_ASSIGN(InstructionVisitor);
 };
 
 
 }  // namespace dart
 
 #endif  // VM_INTERMEDIATE_LANGUAGE_H_