Skip to content

Commit 5a60e0d

Browse files
bnoordhuisColin Ihrig
authored and
Colin Ihrig
committed
v8: back-port JitCodeEvent patch from upstream
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 <cjihrig@gmail.com> PR-URL: nodejs/node-v0.x-archive#25588
1 parent 5fc4fb1 commit 5a60e0d

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

deps/v8/src/log.cc

+5-1
Original file line numberDiff line numberDiff line change
@@ -1485,7 +1485,11 @@ void Logger::LogCodeObject(Object* object) {
14851485
tag = Logger::STUB_TAG;
14861486
break;
14871487
case Code::BUILTIN:
1488-
description = "A builtin from the snapshot";
1488+
description =
1489+
Isolate::Current()->builtins()->Lookup(code_object->entry());
1490+
if (description == NULL) {
1491+
description = "A builtin from the snapshot";
1492+
}
14891493
tag = Logger::BUILTIN_TAG;
14901494
break;
14911495
case Code::KEYED_LOAD_IC:

0 commit comments

Comments
 (0)