Skip to content

honor per-model packed-LHS disable in XNNPACK (#24245) - #24245

Merged
meta-codesync[bot] merged 1 commit into
pytorch:mainfrom
JakeStevens:export-D117888679
Oct 30, 2026
Merged

meta-codesync[bot] merged 1 commit into
pytorch:mainfrom
JakeStevens:export-D117888679

Conversation

@JakeStevens

@JakeStevens JakeStevens commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Add the disable_packed_lhs ExecuTorch XNNPACK backend option with a false global default and per-model runtime-spec overrides. Pass the resolved settings through XNNCompilerOptions. When the internal XNNPACK patch is available, set XNN_FLAG_NO_PACKED_LHS to preserve explicit dynamic quantization; builds without that flag retain their normal runtime flags and log enabled requests as unsupported.

Reviewed By: digantdesai

Differential Revision: D117888679

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 28, 2026
@pytorch-bot

pytorch-bot Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ You can merge normally! (2 Unrelated Failures)

As of commit 7081e1f with merge base 14b5e1d (image):

BROKEN TRUNK - The following jobs failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

@meta-codesync

meta-codesync Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

@JakeStevens has exported this pull request. If you are a Meta employee, you can view the originating Diff in D117888679.

@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.

@meta-codesync meta-codesync Bot changed the title honor per-model packed-LHS disable in XNNPACK honor per-model packed-LHS disable in XNNPACK (#24245) Oct 28, 2026
JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Oct 28, 2026
Summary:

Add the `disable_packed_lhs` ExecuTorch XNNPACK backend option with a false global default and per-model runtime-spec overrides. Pass the resolved settings through `XNNCompilerOptions`. When the internal XNNPACK patch is available, set `XNN_FLAG_NO_PACKED_LHS` to preserve explicit dynamic quantization; builds without that flag retain their normal runtime flags and log enabled requests as unsupported.

Implementation and tests updated with Codex.

Differential Revision: D117888679

@digantdesai digantdesai 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.

Review automatically exported from Phabricator review in Meta.

Summary:

Add the `disable_packed_lhs` ExecuTorch XNNPACK backend option with a false global default and per-model runtime-spec overrides. Pass the resolved settings through `XNNCompilerOptions`. When the internal XNNPACK patch is available, set `XNN_FLAG_NO_PACKED_LHS` to preserve explicit dynamic quantization; builds without that flag retain their normal runtime flags and log enabled requests as unsupported.

Reviewed By: digantdesai

Differential Revision: D117888679
@meta-codesync
meta-codesync Bot merged commit b50c571 into pytorch:main Oct 30, 2026
243 of 245 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants