Re: Short term JSON eval cache (issue 10990076)

6 views
Skip to first unread message

dco...@codeaurora.org

unread,
Sep 29, 2012, 6:27:46 PM9/29/12
to da...@chromium.org, yan...@chromium.org, ul...@chromium.org, v8-...@googlegroups.com
Reviewers: danno, Yang, ulan,

Message:
We mainly focused on seeing performance improvement on Sunspider and
ensuring
that other cases are at least neutral (no degradation) if there is no
noticeable
improvement. Unfortunately for some of these changes we were limited to only
evaluating performance for the benchmarks.

w.r.t. IdleNotification, we need to ensure cache entries are maintained
across
iterations of SunSpider in order for this to be useful. Maybe there's some
other
way to tweak how caches are aged? IdleNotifications trigger a major GC after
every test in SunSpider. What do you think?

Description:
Short term JSON eval cache
BUG=none
TEST=none

Please review this at http://codereview.chromium.org/10990076/

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

Affected files:
M AUTHORS
M src/compilation-cache.h
M src/compilation-cache.cc
M src/compiler.cc
M src/flag-definitions.h
M src/objects-inl.h
M src/objects.h
M src/objects.cc
M src/runtime.h
M src/runtime.cc
M src/v8.cc


Reply all
Reply to author
Forward
0 new messages