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

Issue 1518483002: PPC: [turbofan] Add initial support for SOFT deopts. (Closed)

Created:
5 years ago by MTBrandyberry
Modified:
5 years 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] Add initial support for SOFT deopts. Port 7a22fdff1ebfebcdd83f2cae488915ae8ea2dc81 Original commit message: This only introduces the basic support for both SOFT and EAGER deopts in TurboFan, but doesn't make use of SOFT deopts anywhere yet (there are some other issues blocking that). Basically every Deoptimize operator now has a DeoptimizeKind, which can be either kSoft or kEager. R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= Committed: https://crrev.com/cfef519a1ed03832f47623f3efec4a0617e1ac0f Cr-Commit-Position: refs/heads/master@{#32727}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M src/compiler/ppc/code-generator-ppc.cc View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 11 (4 generated)
MTBrandyberry
5 years ago (2015-12-09 17:56:28 UTC) #2
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1518483002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1518483002/1
5 years ago (2015-12-09 17:56:29 UTC) #3
john.yan
lgtm
5 years ago (2015-12-09 18:10:57 UTC) #4
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years ago (2015-12-09 18:32:47 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1518483002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1518483002/1
5 years ago (2015-12-09 18:34:01 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-12-09 18:35:29 UTC) #9
commit-bot: I haz the power
5 years ago (2015-12-09 18:35:53 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/cfef519a1ed03832f47623f3efec4a0617e1ac0f
Cr-Commit-Position: refs/heads/master@{#32727}

Powered by Google App Engine
This is Rietveld 408576698