[*lev] Inline AsyncFunctionEnter [v8/v8 : main]

0 views
Skip to first unread message

Darius Mercadier (Gerrit)

unread,
Jan 9, 2026, 6:25:15 AM (yesterday) Jan 9
to Victor Gomes, chrom...@appspot.gserviceaccount.com, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
Attention needed from Victor Gomes

Darius Mercadier voted and added 1 comment

Votes added by Darius Mercadier

Auto-Submit+1

1 comment

Patchset-level comments
File-level comment, Patchset 3 (Latest):
Darius Mercadier . resolved

PTAL! :)

Open in Gerrit

Related details

Attention is currently required from:
  • Victor Gomes
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: v8/v8
Gerrit-Branch: main
Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
Gerrit-Change-Number: 7222875
Gerrit-PatchSet: 3
Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
Gerrit-Attention: Victor Gomes <victo...@chromium.org>
Gerrit-Comment-Date: Fri, 09 Jan 2026 11:25:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Victor Gomes (Gerrit)

unread,
Jan 9, 2026, 7:11:14 AM (yesterday) Jan 9
to Darius Mercadier, chrom...@appspot.gserviceaccount.com, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
Attention needed from Darius Mercadier

Victor Gomes added 2 comments

Patchset-level comments
Victor Gomes . resolved

Nice, LGTM. Any idea why we regress Air?

Commit Message
Line 8, Patchset 3 (Latest):
Victor Gomes . unresolved

Add a bug issue

Open in Gerrit

Related details

Attention is currently required from:
  • Darius Mercadier
Submit Requirements:
    • requirement satisfiedCode-Owners
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: v8/v8
    Gerrit-Branch: main
    Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
    Gerrit-Change-Number: 7222875
    Gerrit-PatchSet: 3
    Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
    Gerrit-Attention: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Comment-Date: Fri, 09 Jan 2026 12:11:09 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Victor Gomes (Gerrit)

    unread,
    Jan 9, 2026, 7:11:41 AM (yesterday) Jan 9
    to Darius Mercadier, chrom...@appspot.gserviceaccount.com, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
    Attention needed from Darius Mercadier

    Victor Gomes voted Code-Review+1

    Code-Review+1
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Darius Mercadier
    Submit Requirements:
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: v8/v8
    Gerrit-Branch: main
    Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
    Gerrit-Change-Number: 7222875
    Gerrit-PatchSet: 3
    Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
    Gerrit-Attention: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Comment-Date: Fri, 09 Jan 2026 12:11:35 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Darius Mercadier (Gerrit)

    unread,
    Jan 9, 2026, 7:23:24 AM (yesterday) Jan 9
    to Victor Gomes, chrom...@appspot.gserviceaccount.com, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
    Attention needed from Victor Gomes

    Darius Mercadier voted and added 2 comments

    Votes added by Darius Mercadier

    Auto-Submit+1

    2 comments

    Patchset-level comments
    File-level comment, Patchset 3:
    Victor Gomes . unresolved

    Nice, LGTM. Any idea why we regress Air?

    Darius Mercadier

    Mmmh good question, I'll trigger another pinpoint and if it still repros I'll have a quick look before landing :)
    (marking this as unresolved so that I don't forget before landing)

    Commit Message
    Line 8, Patchset 3:
    Victor Gomes . resolved

    Add a bug issue

    Darius Mercadier

    Done

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Victor Gomes
    Submit Requirements:
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: v8/v8
    Gerrit-Branch: main
    Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
    Gerrit-Change-Number: 7222875
    Gerrit-PatchSet: 4
    Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
    Gerrit-Attention: Victor Gomes <victo...@chromium.org>
    Gerrit-Comment-Date: Fri, 09 Jan 2026 12:23:20 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: Yes
    Comment-In-Reply-To: Victor Gomes <victo...@chromium.org>
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    chromeperf@appspot.gserviceaccount.com (Gerrit)

    unread,
    Jan 9, 2026, 8:24:54 AM (yesterday) Jan 9
    to Darius Mercadier, Victor Gomes, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
    Attention needed from Darius Mercadier and Victor Gomes

    Message from chrom...@appspot.gserviceaccount.com

    📍 Job mac-m1_mini_2020-perf/jetstream-main.crossbench complete.

    See results at: https://pinpoint-dot-chromeperf.appspot.com/job/10eb5cdeb10000

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Darius Mercadier
    • Victor Gomes
    Submit Requirements:
    • requirement satisfiedCode-Owners
    • requirement satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: v8/v8
    Gerrit-Branch: main
    Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
    Gerrit-Change-Number: 7222875
    Gerrit-PatchSet: 4
    Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
    Gerrit-Attention: Darius Mercadier <dmerc...@chromium.org>
    Gerrit-Attention: Victor Gomes <victo...@chromium.org>
    Gerrit-Comment-Date: Fri, 09 Jan 2026 13:24:50 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: No
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    chromeperf@appspot.gserviceaccount.com (Gerrit)

    unread,
    Jan 9, 2026, 8:26:53 AM (yesterday) Jan 9
    to Darius Mercadier, Victor Gomes, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
    Attention needed from Darius Mercadier and Victor Gomes

    Message from chrom...@appspot.gserviceaccount.com

    📍 Job mac-m4-mini-perf/jetstream-main.crossbench complete.

    See results at: https://pinpoint-dot-chromeperf.appspot.com/job/13389519b10000

    Gerrit-Comment-Date: Fri, 09 Jan 2026 13:26:48 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: No
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Darius Mercadier (Gerrit)

    unread,
    Jan 9, 2026, 8:30:26 AM (yesterday) Jan 9
    to Victor Gomes, chrom...@appspot.gserviceaccount.com, V8 LUCI CQ, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org
    Attention needed from Victor Gomes

    Darius Mercadier voted and added 1 comment

    Votes added by Darius Mercadier

    Commit-Queue+2

    1 comment

    Patchset-level comments
    File-level comment, Patchset 3:
    Victor Gomes . resolved

    Nice, LGTM. Any idea why we regress Air?

    Darius Mercadier

    Mmmh good question, I'll trigger another pinpoint and if it still repros I'll have a quick look before landing :)
    (marking this as unresolved so that I don't forget before landing)

    Darius Mercadier

    Air doesn't regress anymore; looks like it was noise.

    async-fs regresses on M1 but improves on M4 so I'm going to call this noise as well and go ahead with landing.

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Victor Gomes
    Submit Requirements:
      • requirement satisfiedCode-Owners
      • requirement satisfiedCode-Review
      • requirement satisfiedReview-Enforcement
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: v8/v8
      Gerrit-Branch: main
      Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
      Gerrit-Change-Number: 7222875
      Gerrit-PatchSet: 4
      Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
      Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
      Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
      Gerrit-Attention: Victor Gomes <victo...@chromium.org>
      Gerrit-Comment-Date: Fri, 09 Jan 2026 13:30:22 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: Yes
      Comment-In-Reply-To: Darius Mercadier <dmerc...@chromium.org>
      Comment-In-Reply-To: Victor Gomes <victo...@chromium.org>
      satisfied_requirement
      open
      diffy

      V8 LUCI CQ (Gerrit)

      unread,
      Jan 9, 2026, 8:32:27 AM (yesterday) Jan 9
      to Darius Mercadier, Victor Gomes, chrom...@appspot.gserviceaccount.com, leszek...@chromium.org, v8-re...@googlegroups.com, verwaes...@chromium.org, victorgo...@chromium.org

      V8 LUCI CQ submitted the change

      Unreviewed changes

      3 is the latest approved patch-set.
      No files were changed between the latest approved patch-set and the submitted one.

      Change information

      Commit message:
      [*lev] Inline AsyncFunctionEnter
      Bug: 431933185
      Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
      Reviewed-by: Victor Gomes <victo...@chromium.org>
      Auto-Submit: Darius Mercadier <dmerc...@chromium.org>
      Commit-Queue: Darius Mercadier <dmerc...@chromium.org>
      Cr-Commit-Position: refs/heads/main@{#104597}
      Files:
      • M src/maglev/maglev-graph-builder.cc
      • M src/maglev/maglev-graph-builder.h
      • M src/maglev/maglev-ir.h
      Change size: M
      Delta: 3 files changed, 143 insertions(+), 6 deletions(-)
      Branch: refs/heads/main
      Submit Requirements:
      • requirement satisfiedCode-Review: +1 by Victor Gomes
      Open in Gerrit
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: merged
      Gerrit-Project: v8/v8
      Gerrit-Branch: main
      Gerrit-Change-Id: I3aecc10b27683d855c6d237dea57d6dec245a8c5
      Gerrit-Change-Number: 7222875
      Gerrit-PatchSet: 5
      Gerrit-Owner: Darius Mercadier <dmerc...@chromium.org>
      Gerrit-Reviewer: Darius Mercadier <dmerc...@chromium.org>
      open
      diffy
      satisfied_requirement
      Reply all
      Reply to author
      Forward
      0 new messages