Rename jni layer cpp files

[mohammed-saalim]

Summary

Renamed JNI layer files to better align with their Java counterparts, improving the name mapping between JNI and Java layers as suggested in #10890.

Changes

• Renamed jni_layer.cpp → jni_module.cpp
• Renamed jni_layer_llama.cpp → jni_llm_module.cpp
• Updated all references in:
  • extension/android/CMakeLists.txt
  • extension/android/jni/BUCK
  • extension/android/jni/selective_jni.buck.bzl
  • extension/android/jni/jni_layer_training.cpp

Testing

• Verified zero remaining references to old filenames
• All build file references updated correctly
• Git history preserved via git mv

Fixes #10890

cc @kirklandsign @cbilgin

[pytorch-bot]

🔗 Helpful Links

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

• 📄 Preview Python docs built from this PR

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

• Rolling out OSDC (ARC) runners on pull & trunk workflows in PyTorch main

❌ 3 New Failures, 2 Unrelated Failures

As of commit e89106e with merge base 89e6416 ():

NEW FAILURES - The following jobs have failed:

• pull / unittest / macos / macos-job (gh)
  export/tests/test_target_recipes.py::TestTargetRecipes::test_ic4_model
• pull / unittest-editable / linux / linux-job (gh)
  examples/models/test/test_export.py::ExportTest::test_ic3_export_to_executorch
• pull / unittest-editable / macos / macos-job (gh)
  export/tests/test_target_recipes.py::TestTargetRecipes::test_vit_model

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 / windows / windows-job (gh) (trunk failure)
  ##[error]The operation was canceled.
• pull / unittest-editable / windows / windows-job (gh) (trunk failure)
  ##[error]The operation was canceled.

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

[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 renames the Android JNI C++ entrypoints to better align file names with their Java counterparts (per #10890), and updates build-system references accordingly.

Changes:

• Renamed the core JNI source file reference from jni_layer.cpp to jni_module.cpp across Buck/CMake.
• Renamed the LLM JNI source file reference from jni_layer_llama.cpp to jni_llm_module.cpp across Buck/CMake.
• Updated internal references/comments to reflect the new filenames.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
extension/android/jni/selective_jni.buck.bzl	Updates selective JNI target to point at jni_module.cpp.
extension/android/jni/jni_module.cpp	New/renamed core JNI implementation file.
extension/android/jni/jni_llm_module.cpp	New/renamed LLM JNI implementation file.
extension/android/jni/jni_layer_training.cpp	Updates comment referencing the renamed core JNI file.
extension/android/jni/BUCK	Updates Buck build targets to use renamed JNI sources and export the new filename.
extension/android/CMakeLists.txt	Updates CMake sources list to use renamed JNI sources.

---

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

[meta-codesync]

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

[mohammed-saalim]

is there something i should work on? @kirklandsign

[Copilot]

Pull request overview

Copilot reviewed 4 out of 6 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.

[kirklandsign]

Could you please fix the lint error?

[nil-is-all]

Hi @mohammed-saalim, thanks for the PR. Please do address the errors and fixes suggested in the lint runner and submit for review again. Thanks!

[mohammed-saalim]

can u check now pls?

[kirklandsign]

Hi @mohammed-saalim also there is a merge conflict

[mohammed-saalim]

Hi @kirklandsign @nil-is-all, sorry for the delay! I've addressed the feedback:

• ✅ Fixed the cmake-format lint error
• ✅ Resolved merge conflicts by rebasing onto latest main

Ready for re-review when you get a chance. Thanks!

[Copilot]

Pull request overview

Copilot reviewed 4 out of 6 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.

[kirklandsign]

@mohammed-saalim sorry new merge conflicts

[nil-is-all]

@mohammed-saalim, sorry to ping again, could you look at the merge conflicts and let us know again?

[mohammed-saalim]

Hey @kirklandsign @nil-is-all, apologies for the delay! Rebased onto latest main, no conflicts now. Ready for re-review. Thanks for your patience! Lemme know if u need something else