Add a new API V8::SetJitCodeEventHandler to push code name and location to users such as profilers.

src/compiler.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 12 matching lines @@
 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #include "v8.h"
 
 #include "compiler.h"
 
 #include "bootstrapper.h"
+#include "code-events.h"
 #include "codegen.h"
 #include "compilation-cache.h"
 #include "debug.h"
 #include "full-codegen.h"
-#include "gdb-jit.h"
 #include "hydrogen.h"
 #include "isolate-inl.h"
 #include "lithium.h"
 #include "liveedit.h"
 #include "parser.h"
 #include "rewriter.h"
 #include "runtime-profiler.h"
 #include "scanner-character-streams.h"
 #include "scopeinfo.h"
 #include "scopes.h"
@@ skipping 438 matching lines @@
   Compiler::SetFunctionInfo(result, lit, true, script);
 
   if (script->name()->IsString()) {
     PROFILE(isolate, CodeCreateEvent(
         info->is_eval()
             ? Logger::EVAL_TAG
             : Logger::ToNativeByScript(Logger::SCRIPT_TAG, *script),
         *info->code(),
         *result,
         String::cast(script->name())));
-    GDBJIT(AddCode(Handle<String>(String::cast(script->name())),
-                   script,
-                   info->code(),
-                   info));
+    JIT_CODE_EVENT(AddCode(Handle<String>(String::cast(script->name())),

[danno, 2012/07/25 13:50:42]

I know this is beyond the scope of your change, but is there any way to unify
this with the PROFILE code events, since that seems to be a bottleneck for all
code creation? 

[Sigurður Ásgeirsson, 2012/07/25 14:38:35]

It really should be possible to do that, though that would probably require
changing or rewriting much of all three mechanisms. There's no question that
there would be maintenance benefits to this, but I don't feel it's something I
can justify taking on for my needs :(.

+                           script,
+                           info->code(),
+                           info));
   } else {
     PROFILE(isolate, CodeCreateEvent(
         info->is_eval()
             ? Logger::EVAL_TAG
             : Logger::ToNativeByScript(Logger::SCRIPT_TAG, *script),
         *info->code(),
         *result,
         isolate->heap()->empty_string()));
-    GDBJIT(AddCode(Handle<String>(), script, info->code(), info));
+    JIT_CODE_EVENT(AddCode(Handle<String>(), script, info->code(), info));
   }
 
   // Hint to the runtime system used when allocating space for initial
   // property space by setting the expected number of properties for
   // the instances of the function.
   SetExpectedNofPropertiesFromEstimate(result, lit->expected_property_count());
 
   script->set_compilation_state(
       Smi::FromInt(Script::COMPILATION_STATE_COMPILED));
 
@@ skipping 501 matching lines @@
                               line_num));
     } else {
       PROFILE(info->isolate(),
               CodeCreateEvent(Logger::ToNativeByScript(tag, *script),
                               *code,
                               *shared,
                               shared->DebugName()));
     }
   }
 
-  GDBJIT(AddCode(Handle<String>(shared->DebugName()),
-                 Handle<Script>(info->script()),
-                 Handle<Code>(info->code()),
-                 info));
+  JIT_CODE_EVENT(AddCode(Handle<String>(shared->DebugName()),
+                         Handle<Script>(info->script()),
+                         Handle<Code>(info->code()),
+                         info));
 }
 
 } }  // namespace v8::internal