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

Issue 2125723002: PPC/s390: [builtins] Unify most of the remaining Math builtins. (Closed)

Created:
4 years, 5 months ago by JaideepBajwa
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

PPC/s390: [builtins] Unify most of the remaining Math builtins. Port 0a0fe8fb8b06d5e90b082838739ebf44cab04028 Original commit message: Import fdlibm versions of acos, acosh, asin and asinh, which are more precise and produce the same result across platforms (we were using libm versions for asin and acos so far, where both speed and precision depended on the operating system so far). Introduce appropriate TurboFan operators for these functions and use them both for inlining and for the generic builtin. Also migrate the Math.imul and Math.fround builtins to TurboFan builtins to ensure that their behavior is always exactly the same as the inlined TurboFan version (i.e. C++ truncation semantics for double to float don't necessarily meet the JavaScript semantics). For completeness, also migrate Math.sign, which can even get some nice love in TurboFan. Drive-by-fix: Some alpha-sorting on the Math related functions, and cleanup the list of Math intrinsics that we have to export via the native context currently. R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG=v8:3266, v8:3496, v8:3509, v8:3952, v8:5169, v8:5170, v8:5171, v8:5172 LOG=N Committed: https://crrev.com/305d42c01022061faf50de0070ae0a6c4b00f739 Cr-Commit-Position: refs/heads/master@{#37539}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -12 lines) Patch
M src/compiler/ppc/code-generator-ppc.cc View 2 chunks +18 lines, -6 lines 0 comments Download
M src/compiler/s390/code-generator-s390.cc View 3 chunks +18 lines, -6 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
JaideepBajwa
PTAL
4 years, 5 months ago (2016-07-05 16:51:31 UTC) #1
john.yan
lgtm
4 years, 5 months ago (2016-07-05 17:26:20 UTC) #2
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/2125723002/1
4 years, 5 months ago (2016-07-05 17:28:58 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-05 18:12:08 UTC) #5
commit-bot: I haz the power
4 years, 5 months ago (2016-07-05 18:14:34 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/305d42c01022061faf50de0070ae0a6c4b00f739
Cr-Commit-Position: refs/heads/master@{#37539}

Powered by Google App Engine
This is Rietveld 408576698