diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2015-06-29 14:36:36 +0200 |
|---|---|---|
| committer | Colin Ihrig <cjihrig@nodejs.org> | 2015-06-29 13:35:22 -0700 |
| commit | 5a60e0d904c38c2bdb04785203b1b784967c870d (patch) | |
| tree | ccf93222e4441abe5a65473196a63a0d12a8ed27 | |
| parent | 5fc4fb108b9fdfdb3057a4b6122edbf67489a4a4 (diff) | |
| download | node-5a60e0d904c38c2bdb04785203b1b784967c870d.tar.gz | |
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: https://github.com/joyent/node/pull/25588
| -rw-r--r-- | deps/v8/src/log.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/deps/v8/src/log.cc b/deps/v8/src/log.cc index b049ffe4e..3ecc20496 100644 --- a/deps/v8/src/log.cc +++ b/deps/v8/src/log.cc @@ -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: |