[turbolev] Support HoleyFloat64Constant in AddVirtualObjectNestedValue [v8/v8 : main]

0 views
Skip to first unread message

Marco Vitale (Gerrit)

unread,
Aug 14, 2026, 5:33:18 AM (yesterday) Aug 14
to Victor Gomes, v8-s...@luci-project-accounts.iam.gserviceaccount.com, dmercadi...@chromium.org, v8-re...@googlegroups.com
Attention needed from Victor Gomes

Marco Vitale voted and added 1 comment

Votes added by Marco Vitale

Auto-Submit+1

1 comment

Patchset-level comments
File-level comment, Patchset 2 (Latest):
Marco Vitale . resolved

First one! PTAL 😂

Open in Gerrit

Related details

Attention is currently required from:
  • Victor Gomes
Submit Requirements:
  • requirement is not 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: I723f572fa6964407481c65bcfe90eded3f35817e
Gerrit-Change-Number: 8250362
Gerrit-PatchSet: 2
Gerrit-Owner: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
Gerrit-Attention: Victor Gomes <victo...@chromium.org>
Gerrit-Comment-Date: Fri, 14 Aug 2026 09:33:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
unsatisfied_requirement
open
diffy

Victor Gomes (Gerrit)

unread,
Aug 14, 2026, 7:26:15 AM (yesterday) Aug 14
to Marco Vitale, v8-s...@luci-project-accounts.iam.gserviceaccount.com, dmercadi...@chromium.org, v8-re...@googlegroups.com
Attention needed from Marco Vitale

Victor Gomes voted

Code-Review+1
Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Marco Vitale
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: I723f572fa6964407481c65bcfe90eded3f35817e
Gerrit-Change-Number: 8250362
Gerrit-PatchSet: 2
Gerrit-Owner: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
Gerrit-Attention: Marco Vitale <mrc...@chromium.org>
Gerrit-Comment-Date: Fri, 14 Aug 2026 11:26:09 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Victor Gomes (Gerrit)

unread,
Aug 14, 2026, 7:26:21 AM (yesterday) Aug 14
to Marco Vitale, v8-s...@luci-project-accounts.iam.gserviceaccount.com, dmercadi...@chromium.org, v8-re...@googlegroups.com
Attention needed from Marco Vitale

Victor Gomes added 1 comment

Patchset-level comments
Victor Gomes . resolved

Thanks!

Open in Gerrit

Related details

Attention is currently required from:
  • Marco Vitale
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: I723f572fa6964407481c65bcfe90eded3f35817e
Gerrit-Change-Number: 8250362
Gerrit-PatchSet: 2
Gerrit-Owner: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
Gerrit-Attention: Marco Vitale <mrc...@chromium.org>
Gerrit-Comment-Date: Fri, 14 Aug 2026 11:26:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy

v8-scoped@luci-project-accounts.iam.gserviceaccount.com (Gerrit)

unread,
Aug 14, 2026, 7:28:46 AM (yesterday) Aug 14
to Marco Vitale, Victor Gomes, dmercadi...@chromium.org, v8-re...@googlegroups.com

v8-s...@luci-project-accounts.iam.gserviceaccount.com submitted the change

Change information

Commit message:
[turbolev] Support HoleyFloat64Constant in AddVirtualObjectNestedValue

AddVirtualObjectNestedValue handles constant nodes when serializing
dematerialized objects into frame state data for Turboshaft deopts.
While IsConstantNode is true for HoleyFloat64Constant, the opcode switch
was missing a case for it, falling through to UNREACHABLE().
Fixed: 546127404
Change-Id: I723f572fa6964407481c65bcfe90eded3f35817e
Reviewed-by: Victor Gomes <victo...@chromium.org>
Auto-Submit: Marco Vitale <mrc...@chromium.org>
Commit-Queue: Victor Gomes <victo...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#109257}
Files:
  • M src/compiler/turboshaft/turbolev-graph-builder.cc
Change size: XS
Delta: 1 file changed, 5 insertions(+), 2 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: I723f572fa6964407481c65bcfe90eded3f35817e
Gerrit-Change-Number: 8250362
Gerrit-PatchSet: 3
Gerrit-Owner: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Marco Vitale <mrc...@chromium.org>
Gerrit-Reviewer: Victor Gomes <victo...@chromium.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages