Arm backend: Add FuseEqualPlaceholdersPass #9893
Conversation
This pass optimizes memory usage by finding constant placeholders pointing to identical tensors and fusing them to one single placeholder with multiple users. Change-Id: Ia125e120b9ba5acb6e91b2e671e6116a0beebaba Signed-off-by: Adrian Lundell <[email protected]>
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/9893
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 17455b4 with merge base a4be2e4 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
facebook-github-bot
added
the
CLA Signed
zingo
added
partner: arm
|
qnn fails are unrelated |
| @@ -61,7 +61,7 @@ tosa_reference_model_rev="70ed0b40fa831387e36abdb4f7fb9670a3464f5a" | |||
|
|
|||
| # vela | |||
| vela_repo_url="https://gitlab.arm.com/artificial-intelligence/ethos-u/ethos-u-vela" | |||
| vela_rev="425541302c7e4b6fbeca7c0061286b131ee507c3" | |||
There was a problem hiding this comment.
Thank you for the heads up @zingo / @AdrianLundell . I think we will pull this one in.
|
@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. |
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