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

Issue 130743006: Merged r16772, r18000, r18298, r18319 into 3.21 branch. (Closed)

Created:
6 years, 11 months ago by ulan
Modified:
6 years, 11 months ago
CC:
v8-dev
Visibility:
Public.

Description

Merged r16772, r18000, r18298, r18319 into 3.21 branch. Add a mechanism to override the detected cpu features. Restore saved caller FP registers on stub failure and preserve FP registers on NotifyStubFailure. Initialize Date parse cache with SMI instead of double to workaround sharing mutable heap numbers in snapshot. Fix NotifyStubFailureSaveDoubles generation for ia32. BUG=280531 LOG=N R=svenpanne@chromium.org

Patch Set 1 #

Patch Set 2 : Add merge of r16772. #

Patch Set 3 : Fix merge #

Patch Set 4 : Fix arm #

Patch Set 5 : Fix x64 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+382 lines, -122 lines) Patch
M src/arguments.h View 1 chunk +11 lines, -0 lines 0 comments Download
M src/arguments.cc View 1 chunk +8 lines, -0 lines 0 comments Download
M src/arm/assembler-arm.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M src/arm/builtins-arm.cc View 3 chunks +13 lines, -3 lines 0 comments Download
M src/arm/deoptimizer-arm.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M src/arm/lithium-codegen-arm.h View 1 chunk +3 lines, -0 lines 0 comments Download
M src/arm/lithium-codegen-arm.cc View 6 chunks +43 lines, -21 lines 0 comments Download
M src/arm/macro-assembler-arm.h View 1 1 chunk +6 lines, -2 lines 0 comments Download
M src/arm/macro-assembler-arm.cc View 1 2 3 2 chunks +7 lines, -4 lines 0 comments Download
M src/assembler.h View 1 1 chunk +13 lines, -0 lines 0 comments Download
M src/assembler.cc View 1 1 chunk +20 lines, -0 lines 0 comments Download
M src/builtins.h View 2 chunks +3 lines, -0 lines 0 comments Download
M src/date.js View 1 chunk +1 line, -1 line 0 comments Download
M src/deoptimizer.h View 1 chunk +4 lines, -0 lines 0 comments Download
M src/deoptimizer.cc View 1 chunk +1 line, -2 lines 0 comments Download
M src/ia32/assembler-ia32.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M src/ia32/builtins-ia32.cc View 3 chunks +18 lines, -2 lines 0 comments Download
M src/ia32/deoptimizer-ia32.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M src/ia32/lithium-codegen-ia32.h View 1 chunk +3 lines, -0 lines 0 comments Download
M src/ia32/lithium-codegen-ia32.cc View 1 2 5 chunks +40 lines, -22 lines 0 comments Download
M src/ia32/macro-assembler-ia32.h View 1 1 chunk +6 lines, -2 lines 0 comments Download
M src/ia32/macro-assembler-ia32.cc View 1 3 chunks +8 lines, -4 lines 0 comments Download
M src/mips/assembler-mips.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M src/mips/builtins-mips.cc View 3 chunks +13 lines, -2 lines 0 comments Download
M src/mips/deoptimizer-mips.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M src/mips/lithium-codegen-mips.h View 1 chunk +3 lines, -0 lines 0 comments Download
M src/mips/lithium-codegen-mips.cc View 6 chunks +43 lines, -21 lines 0 comments Download
M src/mips/macro-assembler-mips.h View 1 1 chunk +6 lines, -2 lines 0 comments Download
M src/mips/macro-assembler-mips.cc View 1 3 chunks +7 lines, -4 lines 0 comments Download
M src/version.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/x64/assembler-x64.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M src/x64/builtins-x64.cc View 3 chunks +13 lines, -2 lines 0 comments Download
M src/x64/deoptimizer-x64.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M src/x64/lithium-codegen-x64.h View 1 chunk +4 lines, -0 lines 0 comments Download
M src/x64/lithium-codegen-x64.cc View 6 chunks +43 lines, -21 lines 0 comments Download
M src/x64/macro-assembler-x64.h View 1 2 3 4 1 chunk +6 lines, -2 lines 0 comments Download
M src/x64/macro-assembler-x64.cc View 1 3 chunks +9 lines, -4 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
ulan
6 years, 11 months ago (2014-01-10 13:05:14 UTC) #1
Sven Panne
LGTM (rubberstamped)
6 years, 11 months ago (2014-01-10 14:06:42 UTC) #2
ulan
Hi Paul, I am merging this CL to 3.21. It fails in mipsel.release.check in many ...
6 years, 11 months ago (2014-01-10 15:28:35 UTC) #3
Paul Lind
Ulan, the fix for this is r18030, which fixes a MIPS-specific code-aging bug. However, it ...
6 years, 11 months ago (2014-01-10 16:37:14 UTC) #4
Paul Lind
If you don't want to merge in 18030 and deal with the conflicts, we can ...
6 years, 11 months ago (2014-01-13 02:10:46 UTC) #5
ulan
On 2014/01/13 02:10:46, Paul Lind wrote: > If you don't want to merge in 18030 ...
6 years, 11 months ago (2014-01-13 08:32:54 UTC) #6
ulan
Abandoning this CL for https://codereview.chromium.org/132193005
6 years, 11 months ago (2014-01-13 09:21:37 UTC) #7
ulan
> Would you like me to include your patch of 18030 in the merge? Or ...
6 years, 11 months ago (2014-01-13 10:12:19 UTC) #8
Paul Lind
6 years, 11 months ago (2014-01-13 17:40:24 UTC) #9
Message was sent while issue was closed.
On 2014/01/13 10:12:19, ulan wrote:
> The 18030 patch adapted for 3.21 also applies cleanly in 3.20. So I'll go
ahead
> and merge it with r16772, r18000, r18298, r18319 to 3.20.
Thanks for taking care of the merges on both these branches, Ulan. I really
appreciate! (Sorry for late reply, it was middle of night here....)

Powered by Google App Engine
This is Rietveld 408576698