Arm backend: Keep internal shared qspec with uint8 IO#20978
Merged
bdemirb merged 1 commit intoJul 17, 2026
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20978
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New FailureAs of commit 1e22c76 with merge base 47fd0d8 ( NEW FAILURE - The following job has failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
meta-cla
Bot
added
the
CLA Signed
Collaborator
Author
|
@pytorchbot label "partner: arm" |
pytorch-bot
Bot
added
the
partner: arm
Collaborator
Author
|
@pytorchbot label "release notes: arm" |
bdemirb
changed the title
robell
approved these changes
Jul 17, 2026
The uint8 IO shared-qspec guard previously removed bridge ops from SharedQspecQuantizer for the whole graph when a model had preserved uint8 IO. That protects image IO accuracy, but it also prevents internal cat clusters from sharing qparams. In RIFE this creates full-resolution FP32/QDQ concat islands before quantized convs. Keep the uint8 IO guard local to shared clusters that actually touch uint8 model IO. Internal shared-qspec clusters can still be annotated and lowered as quantized ops. Add regression coverage for both cases: cat must not bridge uint8 IO to high-range activations, while internal cat between quantized convs must still share qparams. Signed-off-by: Baris Demir <baris.demir@arm.com> Change-Id: I528572d6136f78cbfe27c75579c8b37a4e560887
bdemirb
force-pushed
the
baris/1286838-keep-internal-shared-qspec-uint8
branch
from
July 17, 2026 13:16
17412b2 to
1e22c76
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
cc @digantdesai @freddan80 @per @zingo @oscarandersson8218 @mansnils @Sebastian-Larsson @robell @rascani