Fix executorch/extension/llm/custom_op/...

[mergennachin]

Reviewed By: rascani

Differential Revision: D90888544

[pytorch-bot]

🔗 Helpful Links

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

• 📄 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, 2 Cancelled Jobs, 6 Unrelated Failures

As of commit b7b7add with merge base 9cdc558 ():

NEW FAILURE - The following job has failed:

• Lint / android-java-format / linux-job (gh)

CANCELLED JOBS - The following jobs were cancelled. Please retry:

• pull / test-static-llama-qnn-linux (stories_110m) / linux-job (gh)
  ##[error]The operation was canceled.
• pull / test-static-llama-qnn-linux (stories_260k_bc) / linux-job (gh)
  ##[error]The operation was canceled.

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

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

• pull / unittest / linux / linux-job (gh) (trunk failure)
  backends/test/harness/tests/test_error_statistics.py::ErrorStatisticsTests::test_error_stats_simple
• pull / unittest / macos / macos-job (gh) (trunk failure)
  backends/test/harness/tests/test_error_statistics.py::ErrorStatisticsTests::test_error_stats_simple
• pull / unittest / windows / windows-job (gh) (trunk failure)
  backends/test/harness/tests/test_error_statistics.py::ErrorStatisticsTests::test_error_stats_simple
• pull / unittest-editable / linux / linux-job (gh) (trunk failure)
  backends/test/harness/tests/test_error_statistics.py::ErrorStatisticsTests::test_error_stats_simple
• pull / unittest-editable / macos / macos-job (gh) (trunk failure)
  backends/test/harness/tests/test_error_statistics.py::ErrorStatisticsTests::test_error_stats_simple
• pull / unittest-editable / windows / windows-job (gh) (trunk failure)
  backends/test/harness/tests/test_error_statistics.py::ErrorStatisticsTests::test_error_stats_simple

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

[meta-codesync]

@mergennachin has exported this pull request. If you are a Meta employee, you can view the originating Diff in D90888544.

[github-actions]

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

[Copilot]

Pull request overview

This PR addresses test stability issues by adding threadpool configuration to prevent OMP errors and updating CUDA test requirements to specify minimum compute capability.

Changes:

• Added compute capability check for CUDA/Triton tests requiring CC >= 9.0
• Integrated threadpool reset workaround in test_sdpa_with_kv_cache.py to prevent OMP errors
• Updated TARGETS dependency to include portable_lib for test_sdpa_with_kv_cache

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
extension/llm/custom_ops/test_update_cross_attn_cache.py	Added CUDA compute capability check (CC >= 9.0) for Triton-based tests
extension/llm/custom_ops/test_sdpa_with_kv_cache.py	Added _unsafe_reset_threadpool call to prevent OMP threading errors
extension/llm/custom_ops/TARGETS	Added portable_lib dependency for test_sdpa_with_kv_cache to support threadpool import

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The TARGETS file is missing a corresponding dependency update for test_update_cross_attn_cache. While test_update_cross_attn_cache.py doesn't currently use _unsafe_reset_threadpool, the file is importing from the same module family and may benefit from the same OMP error workaround that was applied to test_sdpa_with_kv_cache and test_quantized_sdpa. Consider adding the dependency if similar threading issues arise in the future.

[meta-codesync]

@mergennachin has imported this pull request. If you are a Meta employee, you can view this in D90888544.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.