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

Issue 1688373002: PPC: [Interpreter] Save and restore dispatch table pointer during calls. (Closed)

Created:
4 years, 10 months ago by MTBrandyberry
Modified:
4 years, 10 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: [Interpreter] Save and restore dispatch table pointer during calls. Port a2935d63dfc943fbf38602c5ce966e8dcb02307a Original commmit message: Saves and restores the dispatch pointer during calls to enable the debugger to switch the dispatch table used by a function during it's execution. Also moves the accumulator and context nodes to be Variables so that they will be properly merged across branches. R=rmcilroy@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG=v8:4280, v8:4690 LOG=N Committed: https://crrev.com/c8257c4c793ef3e3c7b28c73aa765559c3b6998c Cr-Commit-Position: refs/heads/master@{#33962}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -9 lines) Patch
M src/ppc/builtins-ppc.cc View 2 chunks +6 lines, -9 lines 0 comments Download

Messages

Total messages: 15 (6 generated)
MTBrandyberry
4 years, 10 months ago (2016-02-11 23:21:14 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/1688373002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1688373002/1
4 years, 10 months ago (2016-02-11 23:22:02 UTC) #3
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: v8_linux64_avx2_rel on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel/builds/9974)
4 years, 10 months ago (2016-02-11 23:38:07 UTC) #5
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1688373002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1688373002/1
4 years, 10 months ago (2016-02-12 14:28:01 UTC) #7
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 10 months ago (2016-02-12 15:14:46 UTC) #9
michael_dawson
On 2016/02/12 15:14:46, commit-bot: I haz the power wrote: > Dry run: This issue passed ...
4 years, 10 months ago (2016-02-12 23:08:28 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1688373002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1688373002/1
4 years, 10 months ago (2016-02-12 23:10:44 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 10 months ago (2016-02-12 23:12:20 UTC) #13
commit-bot: I haz the power
4 years, 10 months ago (2016-02-12 23:13:04 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/c8257c4c793ef3e3c7b28c73aa765559c3b6998c
Cr-Commit-Position: refs/heads/master@{#33962}

Powered by Google App Engine
This is Rietveld 408576698