Returned checked casts.
http://codereview.chromium.org/3135026/diff/1/6 File src/frames.cc (right): http://codereview.chromium.org/3135026/diff/1/6#newcode410 src/frames.cc:410: return reinterpret_cast<Code*>(code_slot()); On 2010/08/17 10:31:28, Mads Ager wrote:
Doing these changes, we loose assertions in the code that is using
this when not
during GC. Should we add raw_unchecked accessors for the stuff needed
during GC? Done. http://codereview.chromium.org/3135026/diff/1/8 File src/heap-inl.h (right): http://codereview.chromium.org/3135026/diff/1/8#newcode422 src/heap-inl.h:422: return Handle<TYPE>(reinterpret_cast<TYPE*>(__object__)), \ On 2010/08/17 10:31:28, Mads Ager wrote:
Same here, loosing these assertions seems bad.
Agree. This change is unrelated and will be removed. http://codereview.chromium.org/3135026/diff/1/15 File src/objects-visiting.h (right): http://codereview.chromium.org/3135026/diff/1/15#newcode274 src/objects-visiting.h:274: &JSObjectVisitor::template VisitSpecialized<JSFunction::kSize>); On 2010/08/17 10:31:28, Mads Ager wrote:
Long line.
Done. http://codereview.chromium.org/3135026/diff/1/17 File src/objects.h (right): http://codereview.chromium.org/3135026/diff/1/17#newcode3483 src/objects.h:3483: inline int code_age(); On 2010/08/17 10:31:28, Mads Ager wrote:
Add comments please.
Done. http://codereview.chromium.org/3135026/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
