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

Side by Side Diff: src/cpu-profiler-inl.h

Issue 1514006: C++ profiles processor: put under #ifdef and fix issues. (Closed)
Patch Set: Created 10 years, 8 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 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 18 matching lines...) Expand all
29 #define V8_CPU_PROFILER_INL_H_ 29 #define V8_CPU_PROFILER_INL_H_
30 30
31 #include "circular-queue-inl.h" 31 #include "circular-queue-inl.h"
32 #include "profile-generator-inl.h" 32 #include "profile-generator-inl.h"
33 33
34 #include "cpu-profiler.h" 34 #include "cpu-profiler.h"
35 35
36 namespace v8 { 36 namespace v8 {
37 namespace internal { 37 namespace internal {
38 38
39 #ifdef ENABLE_CPP_PROFILES_PROCESSOR
Kevin Millikin (Chromium) 2010/03/30 11:26:29 You could wrap the entire contents of the file exc
mnaganov (inactive) 2010/03/30 11:35:24 Good idea! Fixed.
39 40
40 TickSample* ProfilerEventsProcessor::TickSampleEvent() { 41 TickSample* ProfilerEventsProcessor::TickSampleEvent() {
41 TickSampleEventRecord* evt = 42 TickSampleEventRecord* evt =
42 reinterpret_cast<TickSampleEventRecord*>(ticks_buffer_.Enqueue()); 43 TickSampleEventRecord::cast(ticks_buffer_.Enqueue());
43 evt->order = enqueue_order_; // No increment! 44 evt->order = enqueue_order_; // No increment!
44 return &evt->sample; 45 return &evt->sample;
45 } 46 }
46 47
48 #endif // ENABLE_CPP_PROFILES_PROCESSOR
47 49
48 } } // namespace v8::internal 50 } } // namespace v8::internal
49 51
50 #endif // V8_CPU_PROFILER_INL_H_ 52 #endif // V8_CPU_PROFILER_INL_H_
OLDNEW
« SConstruct ('K') | « src/cpu-profiler.cc ('k') | src/profile-generator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698