Skip to content

Conversation

@AdrianLundell
Copy link
Collaborator

@AdrianLundell AdrianLundell commented Oct 2, 2025

Change-Id: I566e68320531cf7c233764de2a6a7be29e284ec4
Signed-off-by: Adrian Lundell <adrian.lundell@arm.com>
@AdrianLundell AdrianLundell added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: none Do not include this in the release notes labels Oct 2, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 2, 2025

🔗 Helpful Links

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

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

❌ 1 New Failure

As of commit f9c12e6 with merge base 3b16bc1 (image):

NEW FAILURE - The following job has failed:

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

@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 2, 2025
@mergennachin mergennachin requested a review from psiddh October 2, 2025 15:06
@AdrianLundell
Copy link
Collaborator Author

Fails unrelated

@AdrianLundell
Copy link
Collaborator Author

@digantdesai @psiddh Even though this calls scripts in the arm directory I think it makes sense to add now, partly since it is more user friendly to use scripts in the same folder as you are working in and partly since it gives allows for more separation from the arm backend. For example we can add selected aten ops to the executor runner only when building it for the cortex_m backend easily.

@zingo zingo merged commit a9fe0b4 into pytorch:main Oct 8, 2025
129 of 130 checks passed
@digantdesai
Copy link
Contributor

100%. From user point of view, this is an independent backend and different from backends/arm. However, from implementation point of view, we do what makes most sense which is do rely on infrastructure from backends/{arm,xnnpack,transforms} etc. when it makes sense.

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. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants