Skip to content

Conversation

@zingo
Copy link
Collaborator

@zingo zingo commented Nov 19, 2024

New flags on run.sh
--etdump
Build in etdump and profiling, the etdump base64 coded and put in the log
--debug_build
Build debug instead of release
--extra_build_flags
Extra flags to pass to cmake this makes it for example possible to
override the allocator pool size or other build time cmake flags.

The devtools build has been updated so FLATCC_EXECUTABLE can be used to point out the executable as seem to be intended to be possible but was not working.

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 19, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 5193bbb with merge base 0fb2ccf (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 Nov 19, 2024
@zingo zingo added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk labels Nov 19, 2024
@zingo
Copy link
Collaborator Author

zingo commented Nov 19, 2024

@pytorchbot label "topic: not user facing"

@zingo
Copy link
Collaborator Author

zingo commented Nov 19, 2024

@digantdesai This changes files outside Arm folders and need a review from you, Thanks for your time!

using executorch::runtime::internal::EventTracerProfileOpScope;
#define EXECUTORCH_INTERNAL_PROF(NAME) \
EventTracerProfileOpScope event_tracer_op_scope = \
EventTracerProfileOpScope(event_tracer, NAME);
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah AFAICT this is not meant to be used from outside, as you were suspecting.

So I would try using stuff from executorch/runtime/core/event_tracer.h like EventTracer::[start|stop]_profiling?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes that works, PR will be updated soon.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

New flags on run.sh
--etdump
  Build in etdump and profiling, the etdump base64 coded and put in
  the log
--debug_build
  Build debug instead of release
--extra_build_flags
  Extra flags to pass to cmake this makes it for example possible to
  override the allocator pool size or other build time cmake flags.

The devtools build has been updated so FLATCC_EXECUTABLE can be used
to point out the executable.

Signed-off-by: Zingo Andersen <[email protected]>
Change-Id: Ic0fb1e48ee633c5fe91473bdc2db9e894b2fc4fa
@zingo zingo force-pushed the Arm-backend-Add-devtools-support-to-example branch from ed1bdd7 to 3f06cec Compare December 10, 2024 12:04
@zingo
Copy link
Collaborator Author

zingo commented Dec 10, 2024

Error seem unrealted to this patch e.g.

unittest / linux / linux-job] - https://github.com/pytorch/executorch/actions/runs/12255701820/job/34189395827?pr=6959#logs

++ curl --silent --show-error --location --fail --retry 3 --output /tmp/vulkansdk.tar.gz https://sdk.lunarg.com/sdk/download/1.2.198.1/linux/vulkansdk-linux-x86_64-1.2.198.1.tar.gz
curl: (22) The requested URL returned error: 404

@zingo zingo requested a review from digantdesai December 10, 2024 13:37
@freddan80 freddan80 merged commit dabb14e into pytorch:main Dec 17, 2024
107 checks passed
@zingo zingo deleted the Arm-backend-Add-devtools-support-to-example branch December 18, 2024 10:22
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 topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants