Skip to content

Channels last: add the common layout pipeline - #21963

Draft
rascani wants to merge 10 commits into
gh/rascani/22/headfrom
gh/rascani/9/head
Draft

Channels last: add the common layout pipeline#21963
rascani wants to merge 10 commits into
gh/rascani/22/headfrom
gh/rascani/9/head

Conversation

@rascani

@rascani rascani commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Add ToContiguousChannelsLastPass as the shared fixed-point pipeline for
replacing selected operators with channels-last variants and composing existing
data-movement transforms.

Backends can restrict anchors and provide a node-level propagation barrier.
Reporting covers converted anchors and structural-copy counts and bytes; strict
mode rejects skipped anchors, internal copies, and unknown surviving copy sizes.

Un-skip the proof-of-concept model matrix and drive it through the real
pipeline, keeping both the contiguous and the channels-last case sets, and fold
the focused pipeline coverage into the same suite. The channels-last cases pin
that the pipeline leaves an already-channels-last graph alone.

Known gap, tracked separately: the matrix currently ends at 156 permutes against
138 before the pass. The surviving copies are two thirds boundary, so the method
I/O contract is the fix rather than more peephole fusion.

Local Corstone-300 run of the full Cortex-M suite, the transforms suites, and
lintrunner.

Authored with Codex.

[ghstack-poisoned]
@pytorch-bot

pytorch-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown

🔗 Helpful Links

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

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

❌ 1 New Failure, 5 Unrelated Failures

As of commit ccfdfc6 with merge base 8b93850 (image):

NEW FAILURE - The following job has failed:

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.

[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant