Skip to content

[executorch][native-vulkan] Lower rms_norm without a weight on the Vulkan engine - #25097

Open
digantdesai wants to merge 1 commit into
gh/digantdesai/213/basefrom
gh/digantdesai/213/head
Open

digantdesai wants to merge 1 commit into
gh/digantdesai/213/basefrom
gh/digantdesai/213/head

Conversation

@digantdesai

@digantdesai digantdesai commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Stack from ghstack (oldest at bottom):

LowerRMSNorm required a constant weight, so aten.rms_norm with a None weight stayed unlowered and the Vulkan engine rejected it as an unsupported op. Now that et_vk.rms_norm takes an optional weight, the pass also lowers a None weight and passes it through, and the engine binds it as an ET-VK None value. A present weight still has to be a constant of the normalized shape and the input dtype.

The pass now also leaves a norm over a zero-width last dim unlowered, with or without a weight, so it fails as an unsupported op instead of dividing by zero when ET-VK computes the kernel's work grid.

Authored with Claude Code.

Differential Revision: D131119350

NOTE FOR REVIEWERS: This PR has internal Meta-specific changes or comments, please review them on Phabricator!

[ghstack-poisoned]
@pytorch-bot pytorch-bot Bot added the module: vulkan Issues related to the Vulkan delegate and code under backends/vulkan/ label Nov 14, 2026
@pytorch-bot

pytorch-bot Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/25097

Note: Links to docs will display an error until the docs builds have been completed.

❌ 3 New Failures

As of commit 8da889e with merge base 925c1bd (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@github-actions

Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

This branch was previously deployed

1 inactive deployment
cadence — 8da889e5 Deployed Nov 14, 2026 by digantdesai via hifi-op-test / hifi4 #38412
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. meta-exported module: vulkan Issues related to the Vulkan delegate and code under backends/vulkan/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant