[WebNN] Only limit int32 input for dequantizeLinear for ORT default CPU EP [chromium/src : main]

0 views
Skip to first unread message

Wang, Wei4 (Gerrit)

unread,
Feb 27, 2026, 8:08:43 PM (4 hours ago) Feb 27
to Hu, Ningxin, Miao, Bin, Chromium LUCI CQ, AyeAye, chromium...@chromium.org, Jiewei Qian, blink-...@chromium.org, ipc-securi...@chromium.org
Attention needed from Hu, Ningxin

Wang, Wei4 added 1 comment

Patchset-level comments
File-level comment, Patchset 2 (Latest):
Wang, Wei4 . resolved

PTAL, thanks.

Open in Gerrit

Related details

Attention is currently required from:
  • Hu, Ningxin
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • 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: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: Idd8baaee12f20c2ace07ba49a34f445f0d9692d0
Gerrit-Change-Number: 7616747
Gerrit-PatchSet: 2
Gerrit-Owner: Wang, Wei4 <wei4...@intel.com>
Gerrit-Reviewer: Hu, Ningxin <ningx...@intel.com>
Gerrit-Reviewer: Wang, Wei4 <wei4...@intel.com>
Gerrit-CC: Jiewei Qian <q...@chromium.org>
Gerrit-CC: Miao, Bin <bin....@intel.com>
Gerrit-Attention: Hu, Ningxin <ningx...@intel.com>
Gerrit-Comment-Date: Sat, 28 Feb 2026 01:08:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Hu, Ningxin (Gerrit)

unread,
Feb 27, 2026, 8:40:16 PM (4 hours ago) Feb 27
to Wang, Wei4, Miao, Bin, Chromium LUCI CQ, AyeAye, chromium...@chromium.org, Jiewei Qian, blink-...@chromium.org, ipc-securi...@chromium.org
Attention needed from Wang, Wei4

Hu, Ningxin added 1 comment

Commit Message
Line 14, Patchset 2 (Latest):This CL also makes input and zeroPoint of dequantizeLinear share
the same OpSupportLimits item, because their limitations are same
for different backends after CL[2] is merged.
Hu, Ningxin . unresolved

Does it make sense to have a separate CL for this change? That would make this CL focusing on ORT backend change.

Open in Gerrit

Related details

Attention is currently required from:
  • Wang, Wei4
Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement is not 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: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: Idd8baaee12f20c2ace07ba49a34f445f0d9692d0
    Gerrit-Change-Number: 7616747
    Gerrit-PatchSet: 2
    Gerrit-Owner: Wang, Wei4 <wei4...@intel.com>
    Gerrit-Reviewer: Hu, Ningxin <ningx...@intel.com>
    Gerrit-Reviewer: Wang, Wei4 <wei4...@intel.com>
    Gerrit-CC: Jiewei Qian <q...@chromium.org>
    Gerrit-CC: Miao, Bin <bin....@intel.com>
    Gerrit-Attention: Wang, Wei4 <wei4...@intel.com>
    Gerrit-Comment-Date: Sat, 28 Feb 2026 01:40:10 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Wang, Wei4 (Gerrit)

    unread,
    Feb 27, 2026, 8:45:29 PM (4 hours ago) Feb 27
    to Hu, Ningxin, Miao, Bin, Chromium LUCI CQ, AyeAye, chromium...@chromium.org, Jiewei Qian, blink-...@chromium.org, ipc-securi...@chromium.org
    Attention needed from Hu, Ningxin

    Wang, Wei4 added 1 comment

    Commit Message
    Line 14, Patchset 2 (Latest):This CL also makes input and zeroPoint of dequantizeLinear share
    the same OpSupportLimits item, because their limitations are same
    for different backends after CL[2] is merged.
    Hu, Ningxin . unresolved

    Does it make sense to have a separate CL for this change? That would make this CL focusing on ORT backend change.

    Wang, Wei4

    sure, I will separate the chages.

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Hu, Ningxin
    Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement is not 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: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: Idd8baaee12f20c2ace07ba49a34f445f0d9692d0
    Gerrit-Change-Number: 7616747
    Gerrit-PatchSet: 2
    Gerrit-Owner: Wang, Wei4 <wei4...@intel.com>
    Gerrit-Reviewer: Hu, Ningxin <ningx...@intel.com>
    Gerrit-Reviewer: Wang, Wei4 <wei4...@intel.com>
    Gerrit-CC: Jiewei Qian <q...@chromium.org>
    Gerrit-CC: Miao, Bin <bin....@intel.com>
    Gerrit-Attention: Hu, Ningxin <ningx...@intel.com>
    Gerrit-Comment-Date: Sat, 28 Feb 2026 01:45:23 +0000
    Gerrit-HasComments: Yes
    Gerrit-Has-Labels: No
    Comment-In-Reply-To: Hu, Ningxin <ningx...@intel.com>
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Wang, Wei4 (Gerrit)

    unread,
    Feb 27, 2026, 10:22:23 PM (2 hours ago) Feb 27
    to Hu, Ningxin, Miao, Bin, Chromium LUCI CQ, AyeAye, chromium...@chromium.org, Jiewei Qian, blink-...@chromium.org, ipc-securi...@chromium.org
    Attention needed from Hu, Ningxin

    Wang, Wei4 added 2 comments

    Patchset-level comments
    File-level comment, Patchset 4 (Latest):
    Wang, Wei4 . resolved

    PTAL again, thanks.

    Commit Message
    Line 14, Patchset 2:This CL also makes input and zeroPoint of dequantizeLinear share

    the same OpSupportLimits item, because their limitations are same
    for different backends after CL[2] is merged.
    Hu, Ningxin . resolved

    Does it make sense to have a separate CL for this change? That would make this CL focusing on ORT backend change.

    Wang, Wei4

    sure, I will separate the chages.

    Wang, Wei4

    Done

    Open in Gerrit

    Related details

    Attention is currently required from:
    • Hu, Ningxin
    Submit Requirements:
      • requirement satisfiedCode-Coverage
      • 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: chromium/src
      Gerrit-Branch: main
      Gerrit-Change-Id: Idd8baaee12f20c2ace07ba49a34f445f0d9692d0
      Gerrit-Change-Number: 7616747
      Gerrit-PatchSet: 4
      Gerrit-Owner: Wang, Wei4 <wei4...@intel.com>
      Gerrit-Reviewer: Hu, Ningxin <ningx...@intel.com>
      Gerrit-Reviewer: Wang, Wei4 <wei4...@intel.com>
      Gerrit-CC: Jiewei Qian <q...@chromium.org>
      Gerrit-CC: Miao, Bin <bin....@intel.com>
      Gerrit-Attention: Hu, Ningxin <ningx...@intel.com>
      Gerrit-Comment-Date: Sat, 28 Feb 2026 03:22:17 +0000
      Gerrit-HasComments: Yes
      Gerrit-Has-Labels: No
      Comment-In-Reply-To: Hu, Ningxin <ningx...@intel.com>
      Comment-In-Reply-To: Wang, Wei4 <wei4...@intel.com>
      satisfied_requirement
      unsatisfied_requirement
      open
      diffy
      Reply all
      Reply to author
      Forward
      0 new messages