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

Issue 2127683002: Version 5.3.332.9 (cherry-pick) (Closed)

Created:
4 years, 5 months ago by zhengxing.li
Modified:
4 years, 5 months ago
Reviewers:
Michael Hablich
CC:
v8-reviews_googlegroups.com, v8-merges_googlegroups.com, Paweł Hajdan Jr., ulan
Base URL:
https://chromium.googlesource.com/v8/v8.git@5.3
Target Ref:
refs/pending/branch-heads/5.3
Project:
v8
Visibility:
Public.

Description

Version 5.3.332.9 (cherry-pick) Merged d944015623582bf3c8f7e25fcb1e400a315311b9 Merged 43d0b7e7650baf37fc499461b9cd9646cac767a5 Merged 90fa326a6cb97c6465b9252753b61c28c0426286 Merged 7a02c7282f5b753efdeaa0bbbc90768a2b1ad89e Merged 9a9ffd1370dbf91c7f56c0b556a89fb5701c2201 Merged 21550e025ca6ff3a5779350148da037281dc2784 Merged bf4ef548ccc6852e33b0b0bd30ad101a243f8ca6 X87: [builtins] Introduce proper Float64Tan operator. X87: [cleanup] Remove dead code from DeclareLookupSlot and rename it. X87: [builtins] NonNumberToNumber and StringToNumber now use CallRuntime instead of TailCallRuntime. X87: Reland: [Crankshaft] Always check for stubs marked to not require an eager frame. X87: disable some sin/cos/expm1/tan test cases for x87. X87: [RegisterConfiguration] Streamline access to arch defaults, simplify Registers. X87: disable test-gap-resolver/FuzzResolver test case for x87. R=hablich@chromium.org BUG= LOG=N NOTRY=true NOPRESUBMIT=true

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -60 lines) Patch
M include/v8-version.h View 1 chunk +1 line, -1 line 0 comments Download
M src/code-stubs-hydrogen.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M src/compiler/x87/code-generator-x87.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M src/crankshaft/x87/lithium-codegen-x87.cc View 1 chunk +0 lines, -15 lines 0 comments Download
M src/crankshaft/x87/lithium-x87.h View 2 chunks +0 lines, -15 lines 0 comments Download
M src/crankshaft/x87/lithium-x87.cc View 1 chunk +0 lines, -7 lines 0 comments Download
M src/full-codegen/x87/full-codegen-x87.cc View 2 chunks +4 lines, -9 lines 0 comments Download
M src/x87/builtins-x87.cc View 2 chunks +18 lines, -9 lines 0 comments Download
M src/x87/code-stubs-x87.h View 1 chunk +2 lines, -2 lines 0 comments Download
M test/cctest/cctest.status View 1 chunk +5 lines, -0 lines 0 comments Download
M test/unittests/unittests.status View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (3 generated)
zhengxing.li
4 years, 5 months ago (2016-07-06 03:14:10 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2127683002/1
4 years, 5 months ago (2016-07-06 03:15:16 UTC) #3
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
4 years, 5 months ago (2016-07-06 03:15:17 UTC) #5
Michael Hablich
On 2016/07/06 03:15:17, commit-bot: I haz the power wrote: > No L-G-T-M from a valid ...
4 years, 5 months ago (2016-07-06 07:41:10 UTC) #6
zhengxing.li
On 2016/07/06 07:41:10, Hablich wrote: > On 2016/07/06 03:15:17, commit-bot: I haz the power wrote: ...
4 years, 5 months ago (2016-07-06 07:44:23 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2127683002/1
4 years, 5 months ago (2016-07-06 07:44:38 UTC) #9
commit-bot: I haz the power
4 years, 5 months ago (2016-07-06 07:45:02 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698