Skip to content

Commit

Permalink
v8: back-port JitCodeEvent patch from upstream
Browse files Browse the repository at this point in the history
Original commit log follows:

    Meaningful name for builtins in JitCodeEvent API.

    Report builtins by name (e.g. "Builtin:ArgumentsAdaptorTrampoline")
    instead of labeling everything "Builtin:A builtin from the snapshot"

    Review URL: https://codereview.chromium.org/1216833002

Reviewed-By: Colin Ihrig <[email protected]>
PR-URL: nodejs/node-v0.x-archive#25588

Signed-off-by: Jeroen Ooms <[email protected]>
  • Loading branch information
bnoordhuis authored and jeroen committed Jul 7, 2016
1 parent 945e1f1 commit c4fc195
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/log.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1485,7 +1485,11 @@ void Logger::LogCodeObject(Object* object) {
tag = Logger::STUB_TAG;
break;
case Code::BUILTIN:
description = "A builtin from the snapshot";
description =
Isolate::Current()->builtins()->Lookup(code_object->entry());
if (description == NULL) {
description = "A builtin from the snapshot";
}
tag = Logger::BUILTIN_TAG;
break;
case Code::KEYED_LOAD_IC:
Expand Down

0 comments on commit c4fc195

Please sign in to comment.