refactor no longer needed EXECUTORCH_BUILD_HOST_TARGETS

[bvanderhaar]

Summary

Removes EXECUTORCH_BUILD_HOST_TARGETS per @mergennachin comment. Fixes #9404

Test plan

Ensure executorch and cadence builds successfully:

./install_executorch.sh
mkdir cmake-out
cmake -DCMAKE_TOOLCHAIN_FILE=backends/cadence/cadence.cmake \
    -DCMAKE_INSTALL_PREFIX=cmake-out \
    -DCMAKE_BUILD_TYPE=Debug \
    -DPYTHON_EXECUTABLE=python3 \
    -DEXECUTORCH_BUILD_EXTENSION_RUNNER_UTIL=ON \
    -DEXECUTORCH_BUILD_HOST_TARGETS=ON \
    -DEXECUTORCH_BUILD_EXECUTOR_RUNNER=OFF \
    -DEXECUTORCH_BUILD_PTHREADPOOL=OFF \
    -DEXECUTORCH_BUILD_CPUINFO=OFF \
    -Bcmake-out .
cmake --build cmake-out -j8 --target install --config Debug
cmake -DCMAKE_BUILD_TYPE=Debug \
    -DCMAKE_TOOLCHAIN_FILE=examples/backends/cadence.cmake \
    -DCMAKE_PREFIX_PATH=cmake-out \
    -DMODEL_PATH=add.pte \
    -DNXP_SDK_ROOT_DIR=/home/user_name/nxp-sdk \
    -DNN_LIB_BASE_DIR=/home/user_name/nnlib-hifi4 \
    -Bcmake-out/examples/cadence \
    examples/cadence
cmake --build cmake-out/examples/cadence -j8 -t cadence_executorch_example
ls cmake-xt/*.bin

• Test plan assumes nxp-sdk and nnlib-hifir are in user's home directory and Xtensa compiler is available (env vars set). See cadence docs
• Since these are modifications to the build system, I leave it to discretion if CI builds provide enough risk protection to merge.
• Happy to write tests if needed - I'll need a little direction here if needed. Remove usage of external flatc in builds and scripts #9306 ran ./install_executorch.sh, build_apple_frameworks.sh and build_android_library.sh.

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/10320

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

❌ 2 New Failures, 3 Pending

As of commit 9cd6685 with merge base 334af4a ():

NEW FAILURES - The following jobs have failed:

• Lint / lintrunner / linux-job (gh)
  >>> Lint for backends/cadence/aot/tests/test_replace_ops_passes.py:
• pull / unittest-editable / linux / linux-job (gh)
  backends/xnnpack/test/ops/test_conv1d.py::TestConv1d::test_qs8_conv1d_batchnorm_seq

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[bvanderhaar]

@pytorchbot label "topic: not user facing"

[pytorch-bot]

To add the ciflow label ciflow/trunk please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

[jathu]

LGTM so far. However, given the Cadence backend is the only place we are turning this on — can you include that in your test plan?

[bvanderhaar]

LGTM so far. However, given the Cadence backend is the only place we are turning this on — can you include that in your test plan?

I updated the test plan - I'm waiting on Xtensa compiler to validate this locally. Appreciate the guidance.

[pytorch-bot]

To add the ciflow label ciflow/trunk please first approve the workflows that are awaiting approval (scroll to the bottom of this page).

This helps ensure we don't trigger CI on this PR until it is actually authorized to do so. Please ping one of the reviewers if you do not have access to approve and run workflows.

[jathu]

Thank you @bvanderhaar!

[shoumikhin]

@bvanderhaar please check if these CI failures are related
https://hud.pytorch.org/pytorch/executorch/commit/7783adeb1b45ce928c2526381b46b0e0f0c77ee3

[bvanderhaar]

@bvanderhaar please check if these CI failures are related https://hud.pytorch.org/pytorch/executorch/commit/7783adeb1b45ce928c2526381b46b0e0f0c77ee3

"error: Bundle identifier is missing" is the description of the error, googling suggests this is apple-specific. These changes are unlikely to be the cause. I didn't experience this error when testing.