Arm backend: Add tools for finding pte info#18842
Arm backend: Add tools for finding pte info#18842Erik-Lundell wants to merge 1 commit intopytorch:mainfrom
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/18842
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 13 New Failures, 3 Unrelated FailuresAs of commit 1d48089 with merge base 2eaa16c ( NEW FAILURES - The following jobs have failed:
BROKEN TRUNK - The following jobs 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. |
- Add devtools/pte_tool/pte_info.py for getting all compile specs from a pte - Add backends/arm/scripts/ethosu_pte_info.py for Ethos-U specific info. - Use ethosu_pte_info.py in build_executor_runner to set default parameters. Signed-off-by: Erik Lundell <erik.lundell@arm.com> Change-Id: I7ce38fb7b103b628ea69730e9380dd628b44caf3
| @@ -0,0 +1,149 @@ | |||
| #!/usr/bin/env python3 | |||
digantdesai
left a comment
There was a problem hiding this comment.
I think the original idea was if we have these tools in C++, since we will likely be in Python-free environment, it might be handly. In my mind the analogy is with https://www.gnu.org/software/binutils.
Interesting. I would have expected a user to use tools like these as a part of their aot-flow (generate -> inspect -> re-generate), or when cross-compiling on host. What python-free environment do you have in mind? The way I am reading the discussion, a general pteinspect with subcommands would be preferable, correct? Something like (On a side note, this was done as quick un-planned work. If lots of changes are needed, I'll close the PR and postpone this until we have a proper ask for it 😁) |
cc @Gasoonjia @digantdesai @freddan80 @per @zingo @oscarandersson8218 @mansnils @Sebastian-Larsson @robell