Skip to content

Conversation

zingo
Copy link
Collaborator

@zingo zingo commented Jun 10, 2025

Size to runtime has grown and this started to fail after some code was improved/added in the "Refactor FlatTensor" change: #11499 and we got ~50 Bytes over the limit.

cc @digantdesai @freddan80 @per @oscarandersson8218

Size to runtime has grown and this started to fail.

Signed-off-by: Zingo Andersen <zingo.andersen@arm.com>
Change-Id: I03f76b1c2ef6c47876af150db42bb42949c1dffa
Copy link

pytorch-bot bot commented Jun 10, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 5dcba60 with merge base 954f0e9 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-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 Jun 10, 2025
@zingo zingo added module: arm Issues related to arm backend 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 ciflow/trunk and removed module: arm Issues related to arm backend labels Jun 10, 2025
@zingo zingo mentioned this pull request Jun 10, 2025
@zingo zingo requested review from lucylq and JacobSzwejbka June 10, 2025 09:18
@zingo
Copy link
Collaborator Author

zingo commented Jun 10, 2025

#11499 seem to have bumped the bin size slightly for the cortex-m build, tjis bumps the test threshold (assuming the bigger size was ok)

@zingo zingo requested a review from mergennachin June 10, 2025 09:24
Copy link
Contributor

@digantdesai digantdesai left a comment

Choose a reason for hiding this comment

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

While I understand this change, but it would be nice in the future if we track what caused this to fail.

@zingo
Copy link
Collaborator Author

zingo commented Jun 10, 2025

Good idea There are some code added in #11499 when we started to fail, Ill add a pointer to the commit msg for future tracking.

@zingo zingo merged commit 224036e into pytorch:main Jun 10, 2025
192 checks passed
@zingo zingo deleted the Arm-backend-Bump-cortex-m-size-test-threshold branch June 13, 2025 07:20
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.

4 participants