Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2559733002:
[turbofan] regalloc: avoid redundant range intersections (Closed)
|
Created:
4 years ago by Mircea Trofin Modified:
4 years ago CC:
v8-reviews_googlegroups.com Target Ref:
refs/pending/heads/master Project:
v8 Visibility:
Public. |
Description[turbofan] regalloc: avoid redundant range intersections
When finding conflicts, there's no reason to keep looking for registers that are clearly
not going to be available to a candidate live range.
BUG=v8:5644
Committed: https://crrev.com/43e7d05125e9356ce7d6fc504ba30c325521d383
Cr-Commit-Position: refs/heads/master@{#41602}
Patch Set 1 #Patch Set 2 : [turbofan] regalloc: avoid redundant range intersections #Messages
Total messages: 20 (15 generated)
The CQ bit was checked by mtrofin@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...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
Description was changed from ========== [turbofan] regalloc: avoid redundant range intersections BUG= ========== to ========== [turbofan] regalloc: avoid redundant range intersections BUG=v8:5644 ==========
mtrofin@chromium.org changed reviewers: + bradnelson@chromium.org, titzer@chromium.org
Description was changed from ========== [turbofan] regalloc: avoid redundant range intersections BUG=v8:5644 ========== to ========== [turbofan] regalloc: avoid redundant range intersections When finding conflicts, there's no reason to keep looking for registers that are clearly not going to be available to a candidate live range. BUG=v8:5644 ==========
The CQ bit was checked by mtrofin@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...
On 2016/12/08 18:21:38, Mircea Trofin wrote: lgtm
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by mtrofin@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": 20001, "attempt_start_ts": 1481224441766750,
"parent_rev": "0c2d92994d75a57ec2e3887b0d1435f87bd1b409", "commit_rev":
"6b65acbbbe762f4a1cd77461a114aa17bb701768"}
Message was sent while issue was closed.
Description was changed from ========== [turbofan] regalloc: avoid redundant range intersections When finding conflicts, there's no reason to keep looking for registers that are clearly not going to be available to a candidate live range. BUG=v8:5644 ========== to ========== [turbofan] regalloc: avoid redundant range intersections When finding conflicts, there's no reason to keep looking for registers that are clearly not going to be available to a candidate live range. BUG=v8:5644 ==========
Message was sent while issue was closed.
Committed patchset #2 (id:20001)
Message was sent while issue was closed.
Description was changed from ========== [turbofan] regalloc: avoid redundant range intersections When finding conflicts, there's no reason to keep looking for registers that are clearly not going to be available to a candidate live range. BUG=v8:5644 ========== to ========== [turbofan] regalloc: avoid redundant range intersections When finding conflicts, there's no reason to keep looking for registers that are clearly not going to be available to a candidate live range. BUG=v8:5644 Committed: https://crrev.com/43e7d05125e9356ce7d6fc504ba30c325521d383 Cr-Commit-Position: refs/heads/master@{#41602} ==========
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/43e7d05125e9356ce7d6fc504ba30c325521d383 Cr-Commit-Position: refs/heads/master@{#41602} |
