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

Issue 998503002: MIPS: [turbofan] Unify Math.floor / Math.ceil optimization. (Closed)

Created:
5 years, 9 months ago by balazs.kilvady
Modified:
5 years, 9 months ago
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

MIPS: [turbofan] Unify Math.floor / Math.ceil optimization. Port 022ea7e05779f07b9327bcc2d4709fc80b2aeecb Original commit message: Provide an intrinsic %MathFloor / %_MathFloor that is used to optimize both Math.ceil and Math.floor, and use the JS inlining mechanism to inline Math.ceil into TurboFan code. Although we need to touch code outside of TurboFan to make this work, this does not affect the way we handle Math.ceil and/or Math.floor in CrankShaft, because for CrankShaft the old-style builtin function id based inlining still kicks in first. Once this solution is stabilized, we can use it for Math.floor as well. And once that is settled, we can establish it as the unified way to inline builtins, and get rid of the specialized builtin function id based inlining at some point. Note that "builtin" applies to basically every piece of internal JavaScript/intrinsics based code, so this also applies to the yet to be defined JavaScript based code stubs and handlers. BUG=v8:3953 LOG=n Committed: https://crrev.com/9c8bead966994e54ef23389f1c7bda5a500db18b Cr-Commit-Position: refs/heads/master@{#27117}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+48 lines, -32 lines) Patch
M src/compiler/mips/code-generator-mips.cc View 1 chunk +5 lines, -5 lines 0 comments Download
M src/compiler/mips/instruction-codes-mips.h View 1 chunk +2 lines, -2 lines 0 comments Download
M src/compiler/mips/instruction-selector-mips.cc View 3 chunks +17 lines, -9 lines 0 comments Download
M src/compiler/mips64/code-generator-mips64.cc View 1 chunk +5 lines, -5 lines 0 comments Download
M src/compiler/mips64/instruction-codes-mips64.h View 1 chunk +2 lines, -2 lines 0 comments Download
M src/compiler/mips64/instruction-selector-mips64.cc View 3 chunks +17 lines, -9 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
balazs.kilvady
5 years, 9 months ago (2015-03-10 12:01:32 UTC) #2
dusmil.imgtec
LGTM.
5 years, 9 months ago (2015-03-10 15:06:12 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/998503002/1
5 years, 9 months ago (2015-03-10 15:50:28 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: v8_linux64_asan_rel on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel/builds/2193)
5 years, 9 months ago (2015-03-10 16:14:36 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/998503002/1
5 years, 9 months ago (2015-03-10 16:16:55 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 9 months ago (2015-03-10 16:34:46 UTC) #10
commit-bot: I haz the power
5 years, 9 months ago (2015-03-10 16:35:00 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/9c8bead966994e54ef23389f1c7bda5a500db18b
Cr-Commit-Position: refs/heads/master@{#27117}

Powered by Google App Engine
This is Rietveld 408576698