https://codereview.chromium.org/11274014/diff/18001/src/objects-inl.h File src/objects-inl.h (right):
https://codereview.chromium.org/11274014/diff/18001/src/objects-inl.h#newcode730 src/objects-inl.h:730: // TODO: Is this sufficient? On 2012/11/06 12:27:09, rossberg wrote:
Yeah, I don't think it's worth introducing the extra (sub)type in the
heap_types
list when you cannot check for it anyways.
Okay, I've removed this and completely redone the functions in runtime.cc to minimize the checks they do (also changed the safe ones to not use handles). Let me know if I went overboard there. https://codereview.chromium.org/11274014/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
