Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2626013002:
[turbofan] Enable escape analysis. (Closed)
|
Created:
3 years, 11 months ago by Jarin Modified:
3 years, 10 months ago Reviewers:
Tobias Tebbi CC:
v8-reviews_googlegroups.com, Michael Hablich Target Ref:
refs/pending/heads/master Project:
v8 Visibility:
Public. |
Description[turbofan] Enable escape analysis.
Review-Url: https://codereview.chromium.org/2626013002
Cr-Original-Commit-Position: refs/heads/master@{#42229}
Committed: https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44bab793
Review-Url: https://codereview.chromium.org/2626013002
Cr-Commit-Position: refs/heads/master@{#42770}
Committed: https://chromium.googlesource.com/v8/v8/+/a15e9bfd3063792817b31feaddc8fe5a4d6921c2
Patch Set 1 #
Messages
Total messages: 18 (11 generated)
The CQ bit was checked by jarin@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
jarin@chromium.org changed reviewers: + tebbi@chromium.org
PTAL.
On 2017/01/11 13:02:38, Jarin wrote: > PTAL. lgtm
The CQ bit was unchecked by jarin@chromium.org
The CQ bit was checked by jarin@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch.
Bot data: {"patchset_id": 1, "attempt_start_ts": 1484139963397140, "parent_rev":
"18104fac86ed56ab21843e3e628f33aab92b1c4a", "commit_rev":
"30176976e857d4eb84a93fface84849c44bab793"}
Message was sent while issue was closed.
Description was changed from ========== [turbofan] Enable escape analysis. ========== to ========== [turbofan] Enable escape analysis. Review-Url: https://codereview.chromium.org/2626013002 Cr-Commit-Position: refs/heads/master@{#42229} Committed: https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44b... ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1) as https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44b...
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2660063002/ by jarin@chromium.org. The reason for reverting is: Temporarily turn off escape analysis to get a clean canary..
Message was sent while issue was closed.
Description was changed from ========== [turbofan] Enable escape analysis. Review-Url: https://codereview.chromium.org/2626013002 Cr-Commit-Position: refs/heads/master@{#42229} Committed: https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44b... ========== to ========== [turbofan] Enable escape analysis. Review-Url: https://codereview.chromium.org/2626013002 Cr-Commit-Position: refs/heads/master@{#42229} Committed: https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44b... ==========
The CQ bit was checked by tebbi@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch.
Bot data: {"patchset_id": 1, "attempt_start_ts": 1485775289083560, "parent_rev":
"8b6fa9d519fb4af611d91cc2c896532147ea0bda", "commit_rev":
"a15e9bfd3063792817b31feaddc8fe5a4d6921c2"}
Message was sent while issue was closed.
Description was changed from ========== [turbofan] Enable escape analysis. Review-Url: https://codereview.chromium.org/2626013002 Cr-Commit-Position: refs/heads/master@{#42229} Committed: https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44b... ========== to ========== [turbofan] Enable escape analysis. Review-Url: https://codereview.chromium.org/2626013002 Cr-Original-Commit-Position: refs/heads/master@{#42229} Committed: https://chromium.googlesource.com/v8/v8/+/30176976e857d4eb84a93fface84849c44b... Review-Url: https://codereview.chromium.org/2626013002 Cr-Commit-Position: refs/heads/master@{#42770} Committed: https://chromium.googlesource.com/v8/v8/+/a15e9bfd3063792817b31feaddc8fe5a4d6... ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1) as https://chromium.googlesource.com/v8/v8/+/a15e9bfd3063792817b31feaddc8fe5a4d6... |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
