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

Issue 1431923003: PPC: [turbofan] Avoid unnecessary write barriers and improve code generation. (Closed)

Created:
5 years, 1 month ago by MTBrandyberry
Modified:
5 years, 1 month 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: [turbofan] Avoid unnecessary write barriers and improve code generation. Port 1e2770123bd5c1cf7631bf11be846f0fbd884c08 Original commit message: Avoid write barriers when storing values in the root set, and use cheaper write barriers for storing maps or tagged pointers. Also improve the generated code for write barriers, utilizing the out of line code mechanism that is available to TurboFan backends, which moves the unlikely case out of the hot path. R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, dstence@us.ibm.com BUG= Committed: https://crrev.com/e2f4f04f20ac066e935ab5a95ba35009b430188d Cr-Commit-Position: refs/heads/master@{#31927}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+132 lines, -70 lines) Patch
M src/compiler/ppc/code-generator-ppc.cc View 4 chunks +59 lines, -18 lines 0 comments Download
M src/compiler/ppc/instruction-codes-ppc.h View 1 chunk +1 line, -2 lines 0 comments Download
M src/compiler/ppc/instruction-selector-ppc.cc View 1 chunk +72 lines, -50 lines 0 comments Download

Messages

Total messages: 11 (3 generated)
MTBrandyberry
5 years, 1 month ago (2015-11-10 19:36:35 UTC) #1
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1431923003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1431923003/1
5 years, 1 month ago (2015-11-10 19:36:51 UTC) #3
michael_dawson
On 2015/11/10 19:36:51, commit-bot: I haz the power wrote: > Dry run: CQ is trying ...
5 years, 1 month ago (2015-11-10 19:55:10 UTC) #4
michael_dawson
On 2015/11/10 19:36:51, commit-bot: I haz the power wrote: > Dry run: CQ is trying ...
5 years, 1 month ago (2015-11-10 19:55:57 UTC) #5
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 1 month ago (2015-11-10 19:56:13 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1431923003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1431923003/1
5 years, 1 month ago (2015-11-10 19:57:54 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 1 month ago (2015-11-10 19:59:19 UTC) #10
commit-bot: I haz the power
5 years, 1 month ago (2015-11-10 19:59:29 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/e2f4f04f20ac066e935ab5a95ba35009b430188d
Cr-Commit-Position: refs/heads/master@{#31927}

Powered by Google App Engine
This is Rietveld 408576698