Skip to content

Conversation

@AdrianLundell
Copy link
Collaborator

@AdrianLundell AdrianLundell commented Sep 22, 2025

  • Use FetchContent_MakeAvailable to define cmsis-nn target
  • Remove custom MVE definition. Instead use the one defined in CMSIS-NN

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

- Use FetchContent_MakeAvailable to define cmsis-nn target
- Remove custom MVE definition. Instead use the one defined in CMSIS-NN

Signed-off-by: Adrian Lundell <adrian.lundell@arm.com>
Change-Id: Iaa670863f7d041b2a02d84c1d0ebbd11ca4f4568
@AdrianLundell AdrianLundell added the partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm label Sep 22, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 22, 2025

🔗 Helpful Links

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

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

❌ 4 New Failures, 1 Unrelated Failure

As of commit c3a5be1 with merge base 5ae40f1 (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following job 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.

@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 Sep 22, 2025
@AdrianLundell AdrianLundell added the release notes: none Do not include this in the release notes label Sep 23, 2025
@AdrianLundell
Copy link
Collaborator Author

Fails does not look related

@zingo
Copy link
Collaborator

zingo commented Sep 23, 2025

Arm fail is also unrelated
trunk / test-arm-ootb-linux / linux-job
Work is ongoing to fix it in another PR

@zingo zingo merged commit 0329a8a into pytorch:main Sep 23, 2025
296 of 303 checks passed
FATAL_ERROR
"CMSIS-NN target not found. Check your CMSIS_NN_LOCAL_PATH or network connection."
)
FetchContent_MakeAvailable(cmsis_nn)
Copy link
Contributor

Choose a reason for hiding this comment

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

+1

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