Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(113)

Issue 10020040: Merged r11194, r11198, r11201, r11214, r11225, r11233, r11240 into 3.9 branch. (Closed)

Created:
8 years, 8 months ago by danno
Modified:
8 years, 8 months ago
Reviewers:
fschneider
CC:
v8-dev
Visibility:
Public.

Description

Merged r11194, r11198, r11201, r11214, r11225, r11233, r11240 into 3.9 branch. Ensure that arguments object is materialized when deoptimizing from inlined function. Fix performance regressions due to lazy initialization. Fix broken build on Windows due to r11198. Properly support shrinking arrays in CopyDictionaryToObjectElements. Ensure default isolate is present in Add/RemoveCallCompletedCallback. Ensure default isolate is present in Context::New(). MIPS: Ensure proper alignment of LazyInstance objects. BUG=v8:2045, chromium:121407, chromium:118686 R=fschneider@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=11276

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+376 lines, -117 lines) Patch
M src/api.cc View 6 chunks +6 lines, -3 lines 0 comments Download
M src/arm/lithium-arm.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M src/elements.cc View 2 chunks +8 lines, -2 lines 0 comments Download
M src/hydrogen.cc View 1 chunk +2 lines, -1 line 0 comments Download
M src/hydrogen-instructions.h View 3 chunks +7 lines, -2 lines 0 comments Download
M src/ia32/lithium-ia32.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M src/isolate.h View 4 chunks +20 lines, -12 lines 0 comments Download
M src/isolate.cc View 8 chunks +43 lines, -75 lines 0 comments Download
M src/lazy-instance.h View 6 chunks +52 lines, -9 lines 0 comments Download
M src/mips/lithium-mips.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M src/platform.h View 3 chunks +8 lines, -2 lines 0 comments Download
M src/platform-cygwin.cc View 2 chunks +7 lines, -0 lines 0 comments Download
M src/platform-freebsd.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M src/platform-linux.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M src/platform-macos.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M src/platform-nullos.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M src/platform-openbsd.cc View 2 chunks +8 lines, -0 lines 0 comments Download
A src/platform-posix.h View 1 chunk +40 lines, -0 lines 0 comments Download
M src/platform-posix.cc View 3 chunks +11 lines, -3 lines 0 comments Download
M src/platform-solaris.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M src/platform-win32.cc View 5 chunks +19 lines, -5 lines 0 comments Download
M src/v8.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M src/version.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/x64/lithium-x64.cc View 1 chunk +3 lines, -0 lines 0 comments Download
A test/mjsunit/regress/regress-121407.js View 1 chunk +40 lines, -0 lines 0 comments Download
A test/mjsunit/regress/regress-2045.js View 1 chunk +49 lines, -0 lines 0 comments Download
M tools/check-static-initializers.sh View 1 chunk +3 lines, -2 lines 0 comments Download
M tools/gyp/v8.gyp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
danno
8 years, 8 months ago (2012-04-11 13:42:30 UTC) #1
fschneider
8 years, 8 months ago (2012-04-11 13:45:50 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698