Use explicit unique targets for vision operators#17527
Use explicit unique targets for vision operators#17527mcremon-meta wants to merge 3 commits intomainfrom
Conversation
Summary: Use explicit, unique cxx_library targets for each hifi4 operator, following the same pattern used by generic operators. Each operator now has its own named target (e.g., `op_add`, `op_softmax`, `op_quantized_conv2d_nchw_out`) with specific dependencies, making the build configuration more maintainable and explicit. Added `# buildifier: keep sorted` annotation to COMMON_DEPS to enforce alphabetical ordering via the linter. Differential Revision: D93517711
Summary: Use explicit, unique cxx_library targets for each fusion_g3 operator, following the same pattern used by generic and hifi operators. Each operator now has its own named target (e.g., `op_add`, `op_softmax`, `op_dequantize`) with specific dependencies, making the build configuration more maintainable and explicit. Added `# buildifier: keep sorted` annotation to COMMON_DEPS to enforce alphabetical ordering via the linter. Differential Revision: D93520739
Summary: Use explicit, unique cxx_library targets for each vision operator, following the same pattern used by generic operators. Each operator now has its own named target (e.g., `op_add`, `op_softmax`, `op_quantized_conv2d_nchw_out`) with specific dependencies, making the build configuration more maintainable and explicit. Added `# buildifier: keep sorted` annotation to COMMON_DEPS to enforce alphabetical ordering via the linter. Differential Revision: D93537671
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/17527
Note: Links to docs will display an error until the docs builds have been completed. ❌ 34 New Failures, 15 Unrelated FailuresAs of commit 225de4d with merge base cfbc158 ( NEW FAILURES - The following jobs have failed:
BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating Diff in D93537671. |
This PR needs a
|
Summary:
Use explicit, unique cxx_library targets for each vision operator, following the same pattern used by generic operators.
Each operator now has its own named target (e.g.,
op_add,op_softmax,op_quantized_conv2d_nchw_out) with specific dependencies, making the build configuration more maintainable and explicit.Added
# buildifier: keep sortedannotation to COMMON_DEPS to enforce alphabetical ordering via the linter.Differential Revision: D93537671