Skip to content

Conversation

@BujSet
Copy link
Contributor

@BujSet BujSet commented Jul 11, 2025

Summary

Platform file for building Arm bare metal executorch applications.

Test plan

Tested against the arm_executor_runner.cpp that is used in the Arm-EthosU tutorial. Was able to successfully build the Arm Executor Runner, and run the simple add model (TODO).

@pytorch-bot
Copy link

pytorch-bot bot commented Jul 11, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Pending

As of commit e9015c7 with merge base 63431bd (image):

NEW FAILURES - The following jobs have failed:

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 Jul 11, 2025
@BujSet
Copy link
Contributor Author

BujSet commented Jul 11, 2025

@digantdesai @zingo Could I get some advice on this PR? I tried making the changes to examples/arm/executor_runner/arm_executor_runner.cpp to use the new file runtime/platform/default/arm_baremetal.hpp, but when I run the setup.sh and run.sh scripts, I'm met with this error:

/home/zephyruser/executorch/examples/arm/executor_runner/arm_executor_runner.cpp:17:10: fatal error: executorch/runtime/platform/defualt/arm_baremetal.hpp: No such file or directory
   17 | #include <executorch/runtime/platform/defualt/arm_baremetal.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

Despite the new header file being moved to the runtime dir in the ET repo, it seems like the setup.sh and run.sh scripts are unable to see this when they try to build executorch. Any ideas why?

@BujSet
Copy link
Contributor Author

BujSet commented Jul 11, 2025

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Jul 11, 2025
@BujSet BujSet force-pushed the arm_baremetal_pal_split branch from 7684489 to e9015c7 Compare July 14, 2025 15:54
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. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants