PPC/S390[wasm-wide-arith]: wide multiplication implementation in turboshaft and liftoff [v8/v8 : main]

0 views
Skip to first unread message

rt-node-js-v8 (Gerrit)

unread,
3:30 PM (6 hours ago) 3:30 PM
to Milad Farazmand, dmercadi...@chromium.org, v8-re...@googlegroups.com, was...@google.com
Attention needed from Milad Farazmand

Message from rt-node-js-v8

LGTM

Open in Gerrit

Related details

Attention is currently required from:
  • Milad Farazmand
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: Id6f20c2ff578bc3a84a53cd17e896f1e8c89d212
Gerrit-Change-Number: 7860302
Gerrit-PatchSet: 3
Gerrit-Owner: Milad Farazmand <mfar...@ibm.com>
Gerrit-CC: rt-node-js-v8 <rt-node-js...@ibm.com>
Gerrit-Attention: Milad Farazmand <mfar...@ibm.com>
Gerrit-Comment-Date: Tue, 19 May 2026 19:30:41 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Milad Farazmand (Gerrit)

unread,
3:35 PM (6 hours ago) 3:35 PM
to John, Jakob Kummerow, rt-node-js-v8, dmercadi...@chromium.org, v8-re...@googlegroups.com, was...@google.com
Attention needed from Jakob Kummerow and John

Milad Farazmand added 1 comment

Patchset-level comments
File-level comment, Patchset 3 (Latest):
Milad Farazmand . resolved

PTAL.

Open in Gerrit

Related details

Attention is currently required from:
  • Jakob Kummerow
  • John
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: Id6f20c2ff578bc3a84a53cd17e896f1e8c89d212
Gerrit-Change-Number: 7860302
Gerrit-PatchSet: 3
Gerrit-Owner: Milad Farazmand <mfar...@ibm.com>
Gerrit-Reviewer: Jakob Kummerow <jkum...@chromium.org>
Gerrit-Reviewer: John <jun...@ibm.com>
Gerrit-CC: rt-node-js-v8 <rt-node-js...@ibm.com>
Gerrit-Attention: John <jun...@ibm.com>
Gerrit-Attention: Jakob Kummerow <jkum...@chromium.org>
Gerrit-Comment-Date: Tue, 19 May 2026 19:35:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Jakob Kummerow (Gerrit)

unread,
4:02 PM (5 hours ago) 4:02 PM
to Milad Farazmand, Jakob Kummerow, John, rt-node-js-v8, dmercadi...@chromium.org, v8-re...@googlegroups.com, was...@google.com
Attention needed from John and Milad Farazmand

Jakob Kummerow voted and added 1 comment

Votes added by Jakob Kummerow

Code-Review+1

1 comment

Patchset-level comments
Jakob Kummerow . resolved

Non-platform-specific files LGTM; rubberstamping the ppc/s390 specific files as usual.

Open in Gerrit

Related details

Attention is currently required from:
  • John
  • Milad Farazmand
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: Id6f20c2ff578bc3a84a53cd17e896f1e8c89d212
Gerrit-Change-Number: 7860302
Gerrit-PatchSet: 3
Gerrit-Owner: Milad Farazmand <mfar...@ibm.com>
Gerrit-Reviewer: Jakob Kummerow <jkum...@chromium.org>
Gerrit-Reviewer: John <jun...@ibm.com>
Gerrit-CC: rt-node-js-v8 <rt-node-js...@ibm.com>
Gerrit-Attention: John <jun...@ibm.com>
Gerrit-Attention: Milad Farazmand <mfar...@ibm.com>
Gerrit-Comment-Date: Tue, 19 May 2026 20:02:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Milad Farazmand (Gerrit)

unread,
4:04 PM (5 hours ago) 4:04 PM
to Jakob Kummerow, John, rt-node-js-v8, dmercadi...@chromium.org, v8-re...@googlegroups.com, was...@google.com
Attention needed from John

Milad Farazmand added 1 comment

Patchset-level comments
Jakob Kummerow . resolved

Non-platform-specific files LGTM; rubberstamping the ppc/s390 specific files as usual.

Milad Farazmand

Thank you for reviewing.

Open in Gerrit

Related details

Attention is currently required from:
  • John
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: Id6f20c2ff578bc3a84a53cd17e896f1e8c89d212
Gerrit-Change-Number: 7860302
Gerrit-PatchSet: 3
Gerrit-Owner: Milad Farazmand <mfar...@ibm.com>
Gerrit-Reviewer: Jakob Kummerow <jkum...@chromium.org>
Gerrit-Reviewer: John <jun...@ibm.com>
Gerrit-CC: rt-node-js-v8 <rt-node-js...@ibm.com>
Gerrit-Attention: John <jun...@ibm.com>
Gerrit-Comment-Date: Tue, 19 May 2026 20:04:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Jakob Kummerow <jkum...@chromium.org>
satisfied_requirement
open
diffy

Milad Farazmand (Gerrit)

unread,
8:26 PM (1 hour ago) 8:26 PM
to Jakob Kummerow, John, rt-node-js-v8, dmercadi...@chromium.org, v8-re...@googlegroups.com, was...@google.com
Attention needed from John

Milad Farazmand added 1 comment

Patchset-level comments
Milad Farazmand . resolved

@junyan Could you take a look as well when you get a sec, thanks.

Open in Gerrit

Related details

Attention is currently required from:
  • John
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: Id6f20c2ff578bc3a84a53cd17e896f1e8c89d212
Gerrit-Change-Number: 7860302
Gerrit-PatchSet: 3
Gerrit-Owner: Milad Farazmand <mfar...@ibm.com>
Gerrit-Reviewer: Jakob Kummerow <jkum...@chromium.org>
Gerrit-Reviewer: John <jun...@ibm.com>
Gerrit-CC: rt-node-js-v8 <rt-node-js...@ibm.com>
Gerrit-Attention: John <jun...@ibm.com>
Gerrit-Comment-Date: Wed, 20 May 2026 00:25:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages