executorch
executorch copied to clipboard
Arm backend: Add FuseEqualPlaceholdersPass
This pass optimizes memory usage by finding constant placeholders pointing to identical tensors and fusing them to one single placeholder with multiple users.
Note that Ethos-U Vela is updated to include a bug fix related to shared buffers.
cc @digantdesai @freddan80 @per @zingo @oscarandersson8218
:link: Helpful Links
:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/9893
- :page_facing_up: Preview Python docs built from this PR
Note: Links to docs will display an error until the docs builds have been completed.
:white_check_mark: No Failures
As of commit 17455b42e486187856725ae4bdfc9722f4368206 with merge base a4be2e4e26284306e03c4b716317eb0fdf8eac09 ():
:green_heart: Looks good so far! There are no failures yet. :green_heart:
This comment was automatically generated by Dr. CI and updates every 15 minutes.
qnn fails are unrelated
@digantdesai How did it go with this? Did you find any problems we should address/fix?
@digantdesai Hi, is it ok to merge this?
@digantdesai has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
Apologies for a delayed response.
Yeah I won't be able to update vela until next week, I pulled this in and if there are no internal failures i.e. the new version of vela is backwards compatible with the one we have internally, we can merge this ASAP.