Commit 0529841
gate fbcode-only TARGETS dirs with is_fbcode() and rename to BUCK
Summary:
Chunk 6 of fbcode/executorch TARGETS->BUCK migration. 32 directories whose
TARGETS files contained fbcode-only macros (`fbcode_macros` loads,
`fbcode_target(`, `python_library`/`python_binary` from fbcode_macros,
etc.). For each:
1. Created a new `targets.bzl` containing the original loads and a
`define_common_targets(is_fbcode = False)` function whose body opens
with `if not is_fbcode: return` and otherwise contains the original
TARGETS rule definitions.
2. Deleted TARGETS and wrote a thin BUCK that calls
`define_common_targets(is_fbcode = is_fbcode())` using
`fbsource_utils.is_fbcode()` for runtime cell detection.
In fbcode the function body runs and the rules are defined as before. In
xplat the function returns early before any fbcode-only macro is invoked,
so the loads (which do resolve in xplat) are harmless and no rules
materialize.
Directories migrated:
- backends/arm/_passes/fb/test
- backends/arm/fb
- backends/arm/fb/tests
- backends/cortex_m
- backends/cortex_m/quantizer
- backends/example
- backends/vulkan/_passes
- backends/webgpu/test
- devtools/backend_debug/tests
- devtools/fb/visualizer
- devtools/inspector/tests
- devtools/intermediate_output_tap/tests
- devtools/pte_tool/tests
- devtools/size_analysis_tool
- examples/llm_pte_finetuning
- examples/models/fb/gemma3n
- examples/models/fb/gemma4/tests
- examples/models/fb/gemma4/validation
- examples/models/fb/llama4/runner/test
- examples/models/gemma4/tests
- examples/models/phi_4_mini
- examples/models/qwen2_5
- examples/models/qwen3
- examples/models/resnet
- examples/qualcomm
- exir/dialects/edge/spec
- exir/dialects/test
- exir/verification
- extension/audio
- extension/audio/fb/test
- extension/llm/custom_ops/spinquant/test/fb
- scripts/fb/size_monitoring
3 dirs were skipped because they already have a `targets.bzl` and need a
manual merge (devtools/etdump/fb/tests, devtools/etdump/tests, exir/fb).
Reviewed By: mzlee
Differential Revision: D1090820441 parent 2a532de commit 0529841
56 files changed
Lines changed: 1163 additions & 763 deletions
File tree
- backends
- cortex_m
- quantizer
- example
- vulkan/_passes
- webgpu/test
- devtools
- backend_debug/tests
- inspector/tests
- intermediate_output_tap/tests
- pte_tool/tests
- size_analysis_tool
- examples
- llm_pte_finetuning
- models
- gemma4/tests
- phi_4_mini
- qwen2_5
- qwen3
- resnet
- qualcomm
- exir
- dialects
- edge/spec
- test
- verification
- extension/audio
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Lines changed: 7 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
7 | 8 | | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
11 | | - | |
| 12 | + | |
| 13 | + | |
12 | 14 | | |
13 | 15 | | |
14 | 16 | | |
| |||
20 | 22 | | |
21 | 23 | | |
22 | 24 | | |
23 | | - | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
| |||
31 | 34 | | |
32 | 35 | | |
33 | 36 | | |
34 | | - | |
| 37 | + | |
| 38 | + | |
35 | 39 | | |
36 | 40 | | |
37 | 41 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
0 commit comments