Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Side by Side Diff: tools/gen-postmortem-metadata.py

Issue 2287173002: Replace CollectVariables with locals(), update callsites to walk locals instead (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: restore undefined handling Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/runtime/runtime-scopes.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 2
3 # 3 #
4 # Copyright 2012 the V8 project authors. All rights reserved. 4 # Copyright 2012 the V8 project authors. All rights reserved.
5 # Redistribution and use in source and binary forms, with or without 5 # Redistribution and use in source and binary forms, with or without
6 # modification, are permitted provided that the following conditions are 6 # modification, are permitted provided that the following conditions are
7 # met: 7 # met:
8 # 8 #
9 # * Redistributions of source code must retain the above copyright 9 # * Redistributions of source code must retain the above copyright
10 # notice, this list of conditions and the following disclaimer. 10 # notice, this list of conditions and the following disclaimer.
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 'value': 'JavaScriptFrameConstants::kFunctionOffset' }, 161 'value': 'JavaScriptFrameConstants::kFunctionOffset' },
162 { 'name': 'off_fp_args', 162 { 'name': 'off_fp_args',
163 'value': 'JavaScriptFrameConstants::kLastParameterOffset' }, 163 'value': 'JavaScriptFrameConstants::kLastParameterOffset' },
164 164
165 { 'name': 'scopeinfo_idx_nparams', 165 { 'name': 'scopeinfo_idx_nparams',
166 'value': 'ScopeInfo::kParameterCount' }, 166 'value': 'ScopeInfo::kParameterCount' },
167 { 'name': 'scopeinfo_idx_nstacklocals', 167 { 'name': 'scopeinfo_idx_nstacklocals',
168 'value': 'ScopeInfo::kStackLocalCount' }, 168 'value': 'ScopeInfo::kStackLocalCount' },
169 { 'name': 'scopeinfo_idx_ncontextlocals', 169 { 'name': 'scopeinfo_idx_ncontextlocals',
170 'value': 'ScopeInfo::kContextLocalCount' }, 170 'value': 'ScopeInfo::kContextLocalCount' },
171 { 'name': 'scopeinfo_idx_ncontextglobals',
172 'value': 'ScopeInfo::kContextGlobalCount' },
173 { 'name': 'scopeinfo_idx_first_vars', 171 { 'name': 'scopeinfo_idx_first_vars',
174 'value': 'ScopeInfo::kVariablePartIndex' }, 172 'value': 'ScopeInfo::kVariablePartIndex' },
175 173
176 { 'name': 'sharedfunctioninfo_start_position_mask', 174 { 'name': 'sharedfunctioninfo_start_position_mask',
177 'value': 'SharedFunctionInfo::kStartPositionMask' }, 175 'value': 'SharedFunctionInfo::kStartPositionMask' },
178 { 'name': 'sharedfunctioninfo_start_position_shift', 176 { 'name': 'sharedfunctioninfo_start_position_shift',
179 'value': 'SharedFunctionInfo::kStartPositionShift' }, 177 'value': 'SharedFunctionInfo::kStartPositionShift' },
180 178
181 { 'name': 'jsarray_buffer_was_neutered_mask', 179 { 'name': 'jsarray_buffer_was_neutered_mask',
182 'value': 'JSArrayBuffer::WasNeutered::kMask' }, 180 'value': 'JSArrayBuffer::WasNeutered::kMask' },
(...skipping 446 matching lines...) Expand 10 before | Expand all | Expand 10 after
629 627
630 out.write(footer); 628 out.write(footer);
631 629
632 if (len(sys.argv) < 4): 630 if (len(sys.argv) < 4):
633 print('usage: %s output.cc objects.h objects-inl.h' % sys.argv[0]); 631 print('usage: %s output.cc objects.h objects-inl.h' % sys.argv[0]);
634 sys.exit(2); 632 sys.exit(2);
635 633
636 load_objects(); 634 load_objects();
637 load_fields(); 635 load_fields();
638 emit_config(); 636 emit_config();
OLDNEW
« no previous file with comments | « src/runtime/runtime-scopes.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698