Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(387)

Side by Side Diff: src/log.cc

Issue 2053523003: Refactor CpuProfiler. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: address alph's comments Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/log.h" 5 #include "src/log.h"
6 6
7 #include <cstdarg> 7 #include <cstdarg>
8 #include <sstream> 8 #include <sstream>
9 9
10 #include "src/bailout-reason.h" 10 #include "src/bailout-reason.h"
11 #include "src/base/platform/platform.h" 11 #include "src/base/platform/platform.h"
12 #include "src/bootstrapper.h" 12 #include "src/bootstrapper.h"
13 #include "src/code-stubs.h" 13 #include "src/code-stubs.h"
14 #include "src/counters.h" 14 #include "src/counters.h"
15 #include "src/deoptimizer.h" 15 #include "src/deoptimizer.h"
16 #include "src/global-handles.h" 16 #include "src/global-handles.h"
17 #include "src/interpreter/bytecodes.h" 17 #include "src/interpreter/bytecodes.h"
18 #include "src/interpreter/interpreter.h" 18 #include "src/interpreter/interpreter.h"
19 #include "src/libsampler/v8-sampler.h" 19 #include "src/libsampler/v8-sampler.h"
20 #include "src/log-inl.h" 20 #include "src/log-inl.h"
21 #include "src/log-utils.h" 21 #include "src/log-utils.h"
22 #include "src/macro-assembler.h" 22 #include "src/macro-assembler.h"
23 #include "src/perf-jit.h" 23 #include "src/perf-jit.h"
24 #include "src/profiler/cpu-profiler-inl.h" 24 #include "src/profiler/cpu-profiler-inl.h"
25 #include "src/profiler/profiler-listener.h"
25 #include "src/runtime-profiler.h" 26 #include "src/runtime-profiler.h"
26 #include "src/string-stream.h" 27 #include "src/string-stream.h"
27 #include "src/vm-state-inl.h" 28 #include "src/vm-state-inl.h"
28 29
29 namespace v8 { 30 namespace v8 {
30 namespace internal { 31 namespace internal {
31 32
32 33
33 #define DECLARE_EVENT(ignore1, name) name, 34 #define DECLARE_EVENT(ignore1, name) name,
34 static const char* kLogEventsNames[CodeEventListener::NUMBER_OF_LOG_EVENTS] = { 35 static const char* kLogEventsNames[CodeEventListener::NUMBER_OF_LOG_EVENTS] = {
(...skipping 712 matching lines...) Expand 10 before | Expand all | Expand 10 after
747 : isolate_(isolate), 748 : isolate_(isolate),
748 ticker_(NULL), 749 ticker_(NULL),
749 profiler_(NULL), 750 profiler_(NULL),
750 log_events_(NULL), 751 log_events_(NULL),
751 is_logging_(false), 752 is_logging_(false),
752 log_(new Log(this)), 753 log_(new Log(this)),
753 perf_basic_logger_(NULL), 754 perf_basic_logger_(NULL),
754 perf_jit_logger_(NULL), 755 perf_jit_logger_(NULL),
755 ll_logger_(NULL), 756 ll_logger_(NULL),
756 jit_logger_(NULL), 757 jit_logger_(NULL),
758 profiler_listener_(new ProfilerListener(isolate)),
757 listeners_(5), 759 listeners_(5),
758 is_initialized_(false) {} 760 is_initialized_(false) {}
759 761
760 Logger::~Logger() { 762 Logger::~Logger() {
761 delete log_; 763 delete log_;
764 delete profiler_listener_;
762 } 765 }
763 766
764 void Logger::addCodeEventListener(CodeEventListener* listener) { 767 void Logger::addCodeEventListener(CodeEventListener* listener) {
765 bool result = isolate_->code_event_dispatcher()->AddListener(listener); 768 bool result = isolate_->code_event_dispatcher()->AddListener(listener);
766 USE(result); 769 USE(result);
767 DCHECK(result); 770 DCHECK(result);
768 } 771 }
769 772
770 void Logger::removeCodeEventListener(CodeEventListener* listener) { 773 void Logger::removeCodeEventListener(CodeEventListener* listener) {
771 isolate_->code_event_dispatcher()->RemoveListener(listener); 774 isolate_->code_event_dispatcher()->RemoveListener(listener);
(...skipping 1086 matching lines...) Expand 10 before | Expand all | Expand 10 after
1858 removeCodeEventListener(jit_logger_); 1861 removeCodeEventListener(jit_logger_);
1859 delete jit_logger_; 1862 delete jit_logger_;
1860 jit_logger_ = NULL; 1863 jit_logger_ = NULL;
1861 } 1864 }
1862 1865
1863 return log_->Close(); 1866 return log_->Close();
1864 } 1867 }
1865 1868
1866 } // namespace internal 1869 } // namespace internal
1867 } // namespace v8 1870 } // namespace v8
OLDNEW
« no previous file with comments | « src/log.h ('k') | src/profiler/cpu-profiler.h » ('j') | src/profiler/cpu-profiler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698