-
Notifications
You must be signed in to change notification settings - Fork 796
[executorch] Remove EXECUTORCH_CLIENTS visibility gating - Kernels #16476
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 - Kernels #16476
Conversation
Remove the EXECUTORCH_CLIENTS buck client list visibility gating from kernel targets. This replaces all visibility blocks containing @EXECUTORCH_CLIENTS with explicit PUBLIC visibility. This is part 3 of a multi-diff stack to remove visibility gating across all of executorch. This diff covers all kernels/*: - kernels/aten - kernels/fb/custom_ops - kernels/optimized - kernels/portable - kernels/prim_ops - kernels/quantized - kernels/test Differential Revision: [D89902027](https://our.internmc.facebook.com/intern/diff/D89902027/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/16476
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (1 Unrelated Failure)As of commit 297a8c1 with merge base 09b5bdb ( 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
|
… Kernels" Remove the EXECUTORCH_CLIENTS buck client list visibility gating from kernel targets. This replaces all visibility blocks containing EXECUTORCH_CLIENTS with explicit PUBLIC visibility. This is part 3 of a multi-diff stack to remove visibility gating across all of executorch. This diff covers all kernels/*: - kernels/aten - kernels/fb/custom_ops - kernels/optimized - kernels/portable - kernels/prim_ops - kernels/quantized - kernels/test Differential Revision: [D89902027](https://our.internmc.facebook.com/intern/diff/D89902027/) [ghstack-poisoned]
605f234
into
gh/GregoryComer/169/base
…16523) This PR was created by the merge bot to help merge the original PR into the main branch. ghstack PR number: #16476 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/169/base ghstack PR head: https://github.com/pytorch/executorch/tree/gh/GregoryComer/169/head Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/GregoryComer/168/orig Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/GregoryComer/169/orig Differential Revision: [D89902027](https://our.internmc.facebook.com/intern/diff/D89902027/) @diff-train-skip-merge --------- Co-authored-by: Gregory Comer <gjcomer@meta.com>
Stack from ghstack (oldest at bottom):
Remove the EXECUTORCH_CLIENTS buck client list visibility gating from kernel
targets. This replaces all visibility blocks containing @EXECUTORCH_CLIENTS
with explicit PUBLIC visibility.
This is part 3 of a multi-diff stack to remove visibility gating across all of executorch.
This diff covers all kernels/*:
Differential Revision: D89902027