Reviewers: danno, Jakob,

Message:
I updated the code according to your comments.

The code to initialize Vtune support is moved to d8.cc.

And the building system changes are updated.

please help to review it.




Description:
This patch is another propagation version of
https://codereview.chromium.org/10824032 patch
which is for "Enables V8 integration with Intel Vtune performance analysis
tool".
And it depends on the first patch https://codereview.chromium.org/11552033

In this patch, we added the JIT code event handler for Vtune. All the code is in
the folder
src/third_party/vtune.

We defined the ENABLE_VTUNE_JIT_INTERFACE macro to enable these Vtune code
compilation.
We call the SetJitCodeEventHandler(...) in v8.cc to make sure it's invokded if
ENABLE_VTUNE_JIT_INTERFACE is defined.

Please review this at https://codereview.chromium.org/11574031/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  Makefile
  build/common.gypi
  src/d8.cc
  src/third_party/vtune/ittnotify_config.h
  src/third_party/vtune/ittnotify_types.h
  src/third_party/vtune/jitprofiling.h
  src/third_party/vtune/jitprofiling.cc
  src/third_party/vtune/vtune-jit.h
  src/third_party/vtune/vtune-jit.cc
  tools/gyp/v8.gyp


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to