C++ profiles processor: put under #ifdef and fix issues.

C++ profiles processor: put under #ifdef and fix issues.

Committed: http://code.google.com/p/v8/source/detail?r=4317

[mnaganov (inactive), 2010-03-30 07:18:44]

Kevin, this is the first patch. Sorry, it might look out of context for you. I'm
fine if you'll just do a high level sanity check.

There will be 2 more patches: one that wires the processor to VM, and another
which adds an API (I previously had discussed API with Mads and Soeren).

[Kevin Millikin (Chromium), 2010-03-30 11:26:29]

LSTM (Look sane to me).

You might want to consider wrapping the #includes also inside the #ifdefs for
ENABLE_CPP_PROFILES_PROCESSOR.  It might cut down compile time a small bit.

http://codereview.chromium.org/1514006/diff/1/2
File SConstruct (right):

http://codereview.chromium.org/1514006/diff/1/2#newcode1
SConstruct:1: # Copyright 2008 the V8 project authors. All rights reserved.
2010.

http://codereview.chromium.org/1514006/diff/1/4
File src/cpu-profiler-inl.h (right):

http://codereview.chromium.org/1514006/diff/1/4#newcode39
src/cpu-profiler-inl.h:39: #ifdef ENABLE_CPP_PROFILES_PROCESSOR
You could wrap the entire contents of the file except the license comment.

[mnaganov (inactive), 2010-03-30 11:35:24]

Thanks!

http://codereview.chromium.org/1514006/diff/1/2
File SConstruct (right):

http://codereview.chromium.org/1514006/diff/1/2#newcode1
SConstruct:1: # Copyright 2008 the V8 project authors. All rights reserved.
On 2010/03/30 11:26:29, Kevin Millikin wrote:
> 2010.

Done.

http://codereview.chromium.org/1514006/diff/1/4
File src/cpu-profiler-inl.h (right):

http://codereview.chromium.org/1514006/diff/1/4#newcode39
src/cpu-profiler-inl.h:39: #ifdef ENABLE_CPP_PROFILES_PROCESSOR
On 2010/03/30 11:26:29, Kevin Millikin wrote:
> You could wrap the entire contents of the file except the license comment.

Good idea! Fixed.