Intel VTune integration for V8/D8

src/d8.cc

Index: src/d8.cc
===================================================================
--- src/d8.cc	(revision 13696)
+++ src/d8.cc	(working copy)
@@ -45,6 +45,10 @@
 #include "../include/v8-testing.h"
 #endif  // V8_SHARED
 
+#ifdef ENABLE_VTUNE_JIT_INTERFACE
+#include "third_party/vtune/vtune-jit.h"
+#endif
+
 #include "d8.h"
 
 #ifndef V8_SHARED
@@ -1905,6 +1909,11 @@
   DumbLineEditor dumb_line_editor(isolate);
   {
     Initialize(isolate);
+#ifdef ENABLE_VTUNE_JIT_INTERFACE
+    V8::SetFlagsFromString("--nocompact_code_space",
+                           i::StrLength("--nocompact_code_space"));

[danno, 2013/02/25 20:25:38]

Can you please move this call into v8::VTuneInitializer::InitializeVTune(),
everybody who calls InitializeVTune will need it and it will go away for all
clients when you support relocation, so it should be hidden in
InitializeVTune().

+    v8::VTuneInitilizer::initilizeVtune();

[danno, 2013/02/25 20:25:38]

Spelling: Initializer. Please capitalize all method names, e.g.
"InitializeVTune()" rather than "initilizeVtune".

+#endif
     Symbols symbols(isolate);
     InitializeDebugger(isolate);