Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2311153002:
[deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition (Closed)
|
Created:
4 years, 3 months ago by Tobias Tebbi Modified:
4 years, 2 months ago Reviewers:
Benedikt Meurer CC:
v8-reviews_googlegroups.com Target Ref:
refs/pending/heads/master Project:
v8 Visibility:
Public. |
Description[deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition
BUG=v8:5330
R=bmeurer@chromium.org
Committed: https://crrev.com/d9c6f517d17f26262d5311972a234530504fb206
Cr-Commit-Position: refs/heads/master@{#39713}
Patch Set 1 #Patch Set 2 : added separators in deoptimization tracing output for full-codegen and ignition #
Messages
Total messages: 19 (13 generated)
Description was changed from ========== added separators in deoptimization tracing output for full-codegen and ignition BUG=5330 R=bmeurer@chromium.org ========== to ========== added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org ==========
Description was changed from ========== added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org ========== to ========== [deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org ==========
lgtm
The CQ bit was checked by tebbi@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.
The CQ bit was checked by tebbi@chromium.org
The CQ bit was unchecked by tebbi@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org Link to the patchset: https://codereview.chromium.org/2311153002/#ps20001 (title: "added separators in deoptimization tracing output for full-codegen and ignition")
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 tebbi@chromium.org
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...
Message was sent while issue was closed.
Description was changed from ========== [deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org ========== to ========== [deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org ==========
Message was sent while issue was closed.
Committed patchset #2 (id:20001)
Message was sent while issue was closed.
Description was changed from ========== [deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org ========== to ========== [deoptimizer] Added separators in deoptimization tracing output for full-codegen and ignition BUG=v8:5330 R=bmeurer@chromium.org Committed: https://crrev.com/d9c6f517d17f26262d5311972a234530504fb206 Cr-Commit-Position: refs/heads/master@{#39713} ==========
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/d9c6f517d17f26262d5311972a234530504fb206 Cr-Commit-Position: refs/heads/master@{#39713} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
