Fix returning too early when setting options

[jathu]

Summary

Fixes a bug where we return too early since we are using a macro. This is only apparent if set_overridable_option is used outside of a top-level CMake file (i.e. different scope)

Test plan

unittest + CI

[pytorch-bot]

🔗 Helpful Links

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

• 📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 4 Unrelated Failures

As of commit de41306 with merge base 0cf2d53 ():

NEW FAILURE - The following job has failed:

• trunk / unittest-nxp-neutron / linux-job (gh)
  RuntimeError: Command docker exec -t f57f82ad57a2aa0910960a9f73f7681c0616fd47ce86b9f8312bd6bed9a64c8f /exec failed with exit code 1

FLAKY - The following job failed but was likely due to flakiness present on trunk:

• trunk / test-huggingface-transformers (HuggingFaceTB/SmolLM2-135M) / linux-job (gh) (detected as infra flaky with no log or failing log classifier)

BROKEN TRUNK - The following jobs failed but was present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

• pull / unittest / linux / linux-job (gh) (trunk failure)
  examples/models/llama/tests/test_export_llama_lib.py::ExportLlamaLibTest::test_has_expected_ops_and_op_counts
• pull / unittest-editable / linux / linux-job (gh) (trunk failure)
  examples/models/llama/tests/test_export_llama_lib.py::ExportLlamaLibTest::test_has_expected_ops_and_op_counts
• trunk / unittest-release / linux / linux-job (gh) (trunk failure)
  examples/models/llama/tests/test_export_llama_lib.py::ExportLlamaLibTest::test_has_expected_ops_and_op_counts

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