Fix CompilerStats info

runtime/vm/isolate.h

 // Copyright (c) 2013, 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_ISOLATE_H_
 #define VM_ISOLATE_H_
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "vm/base_isolate.h"
@@ skipping 19 matching lines @@
 
 // Forward declarations.
 class AbstractType;
 class ApiState;
 class Array;
 class Capability;
 class CHA;
 class Class;
 class Code;
 class CodeIndexTable;
+class CompilerStats;
 class Debugger;
 class DeoptContext;
 class Error;
 class ExceptionHandlers;
 class Field;
 class Function;
 class GrowableObjectArray;
 class HandleScope;
 class HandleVisitor;
 class Heap;
@@ skipping 539 matching lines @@
 
   void set_thread_state(InterruptableThreadState* state) {
     ASSERT((thread_state_ == NULL) || (state == NULL));
     thread_state_ = state;
   }
 
   InterruptableThreadState* thread_state() const {
     return thread_state_;
   }
 
+  CompilerStats* compiler_stats() {
+    return compiler_stats_;
+  }
+
   // Returns the number of sampled threads.
   intptr_t ProfileInterrupt();
 
   VMTagCounters* vm_tag_counters() {
     return &vm_tag_counters_;
   }
 
   uword user_tag() const {
     return user_tag_;
   }
@@ skipping 122 matching lines @@
   int32_t stack_overflow_count_;
   MessageHandler* message_handler_;
   IsolateSpawnState* spawn_state_;
   bool is_runnable_;
   Dart_GcPrologueCallback gc_prologue_callback_;
   Dart_GcEpilogueCallback gc_epilogue_callback_;
   intptr_t defer_finalization_count_;
   DeoptContext* deopt_context_;
   int32_t edge_counter_increment_size_;
 
+  CompilerStats* compiler_stats_;

[siva, 2015/05/12 17:39:52]

Can this be a regular field here instead of a pointer? Is it because you do not
want to pay the cost of this under a flag or in some specific isolates?

[hausner, 2015/05/12 21:52:16]

Changed the TimerScope class so it can handle NULL as timer value. Thus, I only
need to allocate compiler_stats_ if we actually collect the stats.

+
   // Log.
   bool is_service_isolate_;
   class Log* log_;
 
   // Status support.
   char* stacktrace_;
   intptr_t stack_frame_index_;
 
   // Timestamps of last operation via service.
   int64_t last_allocationprofile_accumulator_reset_timestamp_;
@@ skipping 207 matching lines @@
   uint8_t* serialized_args_;
   intptr_t serialized_args_len_;
   uint8_t* serialized_message_;
   intptr_t serialized_message_len_;
   bool paused_;
 };
 
 }  // namespace dart
 
 #endif  // VM_ISOLATE_H_