Skip to content

Revert "Arm backend: Add FuseViewCopyTransform and FuseConstantsPass in arm_pass_manager" - #9070

Closed
zingo wants to merge 2 commits into
mainfrom
revert-8997-change-985586
Closed

Revert "Arm backend: Add FuseViewCopyTransform and FuseConstantsPass in arm_pass_manager"#9070
zingo wants to merge 2 commits into
mainfrom
revert-8997-change-985586

Conversation

@zingo

@zingozingo commented Mar 8, 2025

Copy link
Copy Markdown
Collaborator

@zingo
zingo requested a review from digantdesai as a code ownerMarch 8, 2025 15:00
@pytorch-bot

pytorch-botBot commented Mar 8, 2025

Copy link
Copy Markdown

🔗 Helpful Links

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

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

❌ 2 New Failures, 2 Cancelled Jobs

As of commit 6d75e44 with merge base 51901f3 (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOBS - The following jobs were cancelled. Please retry:

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

@facebook-github-botfacebook-github-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 Mar 8, 2025
@zingozingo added ciflow/trunk partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing labels Mar 8, 2025
@zingo
zingo requested a review from kirklandsignMarch 8, 2025 15:04
@kirklandsign

Copy link
Copy Markdown
Contributor

Maybe we can use #9078 instead. cc @zonglinpeng

@kirklandsign
kirklandsign self-requested a review March 10, 2025 04:24
@zingo
zingo marked this pull request as draft March 10, 2025 12:48
@kirklandsign

Copy link
Copy Markdown
Contributor

We decided to forward fix

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

No longer needed

@zingo

Copy link
Copy Markdown
CollaboratorAuthor

closing

@zingozingo closed this Mar 14, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunkCLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.partner: armFor backend delegation, kernels, demo, etc. from the 3rd-party partner, Armtopic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@zingo@kirklandsign@facebook-github-bot