Skip to content

Commit e3550f2

Browse files
mmarchinitargos
authored andcommitted
deps: backport 958b761 from upstream V8
Original commit message: [postmortem] add postmortem metadata for symbols As discussed in nodejs/llnode#156, we need postmortem metadata for Symbols to properly print Symbol property names in postmortem debugging tools. Patch suggested by Ben Noordhuis (nodejs/llnode#156 (comment)). [email protected], [email protected] Change-Id: Ied6d3c079e8b23a9c796bc632c37785ed7dbc118 Reviewed-on: https://chromium-review.googlesource.com/1205052 Reviewed-by: Benedikt Meurer <[email protected]> Reviewed-by: Yang Guo <[email protected]> Commit-Queue: Yang Guo <[email protected]> Cr-Commit-Position: refs/heads/master@{#55632} Refs: v8/v8@958b761 PR-URL: #22914 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent e5bfab0 commit e3550f2

File tree

4 files changed

+6
-7
lines changed

4 files changed

+6
-7
lines changed

common.gypi

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333

3434
# Reset this number to 0 on major V8 upgrades.
3535
# Increment by one for each non-official patch applied to deps/v8.
36-
'v8_embedder_string': '-node.33',
36+
'v8_embedder_string': '-node.34',
3737

3838
# Enable disassembler for `--print-code` v8 options
3939
'v8_enable_disassembler': 1,

deps/v8/BUILD.gn

+2
Original file line numberDiff line numberDiff line change
@@ -828,6 +828,8 @@ action("postmortem-metadata") {
828828
"src/objects/js-regexp-string-iterator.h",
829829
"src/objects/map.h",
830830
"src/objects/map-inl.h",
831+
"src/objects/name.h",
832+
"src/objects/name-inl.h",
831833
"src/objects/scope-info.h",
832834
"src/objects/script.h",
833835
"src/objects/script-inl.h",

deps/v8/gypfiles/v8.gyp

+2
Original file line numberDiff line numberDiff line change
@@ -2870,6 +2870,8 @@
28702870
'../src/objects/js-regexp-string-iterator.h',
28712871
'../src/objects/map.h',
28722872
'../src/objects/map-inl.h',
2873+
'../src/objects/name.h',
2874+
'../src/objects/name-inl.h',
28732875
'../src/objects/scope-info.h',
28742876
'../src/objects/script.h',
28752877
'../src/objects/script-inl.h',

deps/v8/tools/gen-postmortem-metadata.py

+1-6
Original file line numberDiff line numberDiff line change
@@ -422,15 +422,10 @@ def load_objects_from_file(objfilename, checktypes):
422422
# way around.
423423
#
424424
for type in types:
425-
#
426-
# Symbols and Strings are implemented using the same classes.
427-
#
428-
usetype = re.sub('SYMBOL_', 'STRING_', type);
429-
430425
#
431426
# REGEXP behaves like REG_EXP, as in JS_REGEXP_TYPE => JSRegExp.
432427
#
433-
usetype = re.sub('_REGEXP_', '_REG_EXP_', usetype);
428+
usetype = re.sub('_REGEXP_', '_REG_EXP_', type);
434429

435430
#
436431
# Remove the "_TYPE" suffix and then convert to camel case,

0 commit comments

Comments
 (0)