Skip to content

Conversation

@oscarandersson8218
Copy link
Collaborator

@oscarandersson8218 oscarandersson8218 commented Nov 3, 2025

Merges FP and INT pass pipelines into one pipeline.

cc @freddan80 @per @zingo @digantdesai

Merges FP and INT pass pipelines into one pipeline.

Signed-off-by: Oscar Andersson <oscar.andersson@arm.com>
Change-Id: I42d228d54c46bd9e524c63431830e45a08052a46
@pytorch-bot
Copy link

pytorch-bot bot commented Nov 3, 2025

🔗 Helpful Links

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

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

❌ 1 Cancelled Job

As of commit f8ed445 with merge base a11d555 (image):

CANCELLED JOB - The following job was cancelled. Please retry:

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 Nov 3, 2025
@oscarandersson8218 oscarandersson8218 added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk release notes: none Do not include this in the release notes labels Nov 3, 2025
@zingo
Copy link
Collaborator

zingo commented Nov 3, 2025

@digantdesai @mergennachin This is touching many files but will not add file och move API around so I think it should be fine form buck2 perspective to merge. Ill wait a few day before merging to let any of you have time to stop up if you think it needs internal testing anyway.

@zingo
Copy link
Collaborator

zingo commented Nov 3, 2025

Hi @SS-JIA Im not sure if you want to test this internally or not if so let us know, it just changes small stuff in many files but does not move code around or add/remove files so we consider it safe but just try to alert you when when do "larger" changes.

@SS-JIA
Copy link
Contributor

SS-JIA commented Nov 3, 2025

Should be ok but just to err on the side of caution I will trigger internal test.

Copy link
Contributor

@SS-JIA SS-JIA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having some trouble importing the diff at the moment; but the changes look fairly inocuous so LGTM to me.

@zingo zingo merged commit 7741e70 into pytorch:main Nov 3, 2025
300 of 309 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk 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.

3 participants