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

Issue 2119773003: X87: [turbofan] Introduce Float64Pow and NumberPow operators. (Closed)

Created:
4 years, 5 months ago by zhengxing.li
Modified:
4 years, 5 months ago
CC:
v8-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

X87: [turbofan] Introduce Float64Pow and NumberPow operators. port e607e12ea0a706a0b8cb109e95f923426094dc09 (r37323) original commit message: Introduce a new machine operator Float64Pow that for now is backed by the existing MathPowStub to start the unification of Math.pow, and at the same time address the main performance issue that TurboFan still has with the imaging-darkroom benchmark in Kraken. Also migrate the Math.pow builtin itself to a TurboFan builtin and remove a few hundred lines of hand-written platform code for special handling of the fullcodegen Math.pow version. BUG= Committed: https://crrev.com/727266fcb8b11becf58a298dcd909baa89d2b582 Cr-Commit-Position: refs/heads/master@{#37468}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -18 lines) Patch
M src/compiler/x87/code-generator-x87.cc View 1 chunk +10 lines, -0 lines 0 comments Download
M src/x87/code-stubs-x87.cc View 1 chunk +18 lines, -18 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
zhengxing.li
PTAL, thanks!
4 years, 5 months ago (2016-07-01 02:46:33 UTC) #2
Weiliang
lgtm
4 years, 5 months ago (2016-07-01 03:29:22 UTC) #3
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/2119773003/1
4 years, 5 months ago (2016-07-01 03:30:45 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-01 04:32:11 UTC) #6
commit-bot: I haz the power
4 years, 5 months ago (2016-07-01 04:32:36 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/727266fcb8b11becf58a298dcd909baa89d2b582
Cr-Commit-Position: refs/heads/master@{#37468}

Powered by Google App Engine
This is Rietveld 408576698