Skip to content

[WebNN EP] Support SkipLayerNormalization op - #32377

Merged
Dwayne Robinson (fdwr) merged 2 commits into
microsoft:mainfrom
Honry:support-SkipLayerNormalization
Sep 3, 2026
Merged

Dwayne Robinson (fdwr) merged 2 commits into
microsoft:mainfrom
Honry:support-SkipLayerNormalization

Conversation

@Honry

Copy link
Copy Markdown
Contributor

SkipLayerNormalization can be decomposed by LayerNormalization + Add.

output=LayerNorm(X+skip+bias)

Copilot AI balanced review requested due to automatic review settings September 2, 2026 05:24
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@Honry

Copy link
Copy Markdown
Contributor Author

Dwayne Robinson (@fdwr), PTAL, thanks!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟑 Changes recommended

The implementation uses the wrong default epsilon, incompletely validates its decomposition, and lacks WebNN-specific coverage.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds WebNN support for SkipLayerNormalization via residual addition and native layer normalization.

Changes:

  • Registers and maps the operator.
  • Implements residual/bias handling and optional sum output.
  • Updates operator documentation.
File summaries
File Description
op_builder_factory.cc Registers the operator builder.
map_info.h Adds WebNN operation mapping.
normalization_op_builder.cc Implements decomposition and validation.
webnn-operators.md Documents WebNN support.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 3
  • Review effort level: Balanced

πŸ’‘ Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Honry

Copy link
Copy Markdown
Contributor Author

Copilot, comments addressed, pls. take another look, thanks!

@Honry
Wanming Lin (Honry) force-pushed the support-SkipLayerNormalization branch from c6122b8 to 32f769e Compare September 3, 2026 00:12

@fdwr Dwayne Robinson (fdwr) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ‘

@fdwr
Dwayne Robinson (fdwr) merged commit c5ead59 into microsoft:main Sep 3, 2026
89 of 90 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants