Reviewers: rossberg,

Message:
Hi Andreas,

I'm not sure whether this is worth submitting, but I wanted to pass it along
since I spent a bit of time Friday hacking it up. It'd be mostly superseded by speeding up Object.freeze() in general, but it may be worth it in the long term
since we know exactly the layout of these objects ahead of time.

Description:
Create pre-frozen change records for Object.observe

Added a JSChangeRecord bag-of-constants to objects.h and use it to create two
custom maps for each context in bootstrapper.cc.

This greatly increases performance (~5x) of the ChangeSummary fuzzer (which
tests creating lots of change records).

Before (ms): 4102, 4263, 4219
After (ms): 860, 871, 870


Please review this at https://codereview.chromium.org/12183018/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/bootstrapper.cc
  M src/contexts.h
  M src/heap.h
  M src/object-observe.js
  M src/objects.h
  M src/objects.cc
  M src/runtime.h
  M src/runtime.cc
  M src/v8natives.js


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to