Can someone clarify the context of this CL? Michael is currently on
holidays and
I do not really know what I'm looking at.
The proposed changes seem to alter the semantics of what the script
originally
intended to do. Is this intentional?
http://codereview.chromium.org/10827379/diff/1/tools/gen-postmortem-metadata.py
File tools/gen-postmortem-metadata.py (right):
http://codereview.chromium.org/10827379/diff/1/tools/gen-postmortem-metadata.py#newcode86
tools/gen-postmortem-metadata.py:86: 'value':
'TransitionArray::kFirstIndex' },
All these changes change the semantics of the fields. Is this on
purpose? Do we care about transitions rather than descriptors now?
http://codereview.chromium.org/10827379/diff/1/tools/gen-postmortem-metadata.py#newcode113
tools/gen-postmortem-metadata.py:113: 'Map, instance_descriptors,
uintptr_t, kTransitionsOrBackPointerOffset',
This field does not contain the instance_descriptors but rather either
the transition array or the back pointer. If it contains a transition
array, that array will store the descriptors.
http://codereview.chromium.org/10827379/
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev