-
Notifications
You must be signed in to change notification settings - Fork 795
[executorch] Remove EXECUTORCH_CLIENTS visibility gating - XNNPACK & Vulkan Backends #16477
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[executorch] Remove EXECUTORCH_CLIENTS visibility gating - XNNPACK & Vulkan Backends #16477
Conversation
…Vulkan Backends Remove the EXECUTORCH_CLIENTS buck client list visibility gating from XNNPACK and Vulkan backend targets. This replaces all visibility blocks containing @EXECUTORCH_CLIENTS with explicit PUBLIC visibility. This is part 4 of a multi-diff stack to remove visibility gating across all of executorch. This diff covers: - backends/xnnpack/* - backends/vulkan/* Differential Revision: [D89902028](https://our.internmc.facebook.com/intern/diff/D89902028/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/16477
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New Failure, 1 Unrelated FailureAs of commit d759e28 with merge base 09b5bdb ( NEW FAILURE - The following job has failed:
UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
… XNNPACK & Vulkan Backends" Remove the EXECUTORCH_CLIENTS buck client list visibility gating from XNNPACK and Vulkan backend targets. This replaces all visibility blocks containing EXECUTORCH_CLIENTS with explicit PUBLIC visibility. This is part 4 of a multi-diff stack to remove visibility gating across all of executorch. This diff covers: - backends/xnnpack/* - backends/vulkan/* Differential Revision: [D89902028](https://our.internmc.facebook.com/intern/diff/D89902028/) cc SS-JIA manuelcandales digantdesai cbilgin [ghstack-poisoned]
b28b73e
into
gh/GregoryComer/170/base
…Vulkan Backends (#16524) This PR was created by the merge bot to help merge the original PR into the main branch. ghstack PR number: #16477 by @GregoryComer ^ Please use this as the source of truth for the PR details, comments, and reviews ghstack PR base: https://github.com/pytorch/executorch/tree/gh/GregoryComer/170/base ghstack PR head: https://github.com/pytorch/executorch/tree/gh/GregoryComer/170/head Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/GregoryComer/169/orig Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/GregoryComer/170/orig Differential Revision: [D89902028](https://our.internmc.facebook.com/intern/diff/D89902028/) @diff-train-skip-merge cc @SS-JIA @manuelcandales @digantdesai @cbilgin --------- Co-authored-by: Gregory Comer <[email protected]>
Stack from ghstack (oldest at bottom):
Remove the EXECUTORCH_CLIENTS buck client list visibility gating from XNNPACK
and Vulkan backend targets. This replaces all visibility blocks containing
@EXECUTORCH_CLIENTS with explicit PUBLIC visibility.
This is part 4 of a multi-diff stack to remove visibility gating across all of executorch.
This diff covers:
Differential Revision: D89902028
cc @SS-JIA @manuelcandales @digantdesai @cbilgin