Skip to content

Fix Cadence CPU runner CMake build#18021

Open
aliafzal wants to merge 1 commit intomainfrom
fix-cadence-build
Open

Fix Cadence CPU runner CMake build#18021
aliafzal wants to merge 1 commit intomainfrom
fix-cadence-build

Conversation

@aliafzal
Copy link

@aliafzal aliafzal commented Mar 9, 2026

Summary

  • Detect lib vs lib64 install directory in build_cadence_runner.sh
  • Add -DPYTHON_EXECUTABLE to prevent GLOB_RECURSE scanning entire filesystem when variable is unset
  • Update generic/operators/CMakeLists.txt with actual source filenames

Test plan

  • Built cadence_runner successfully on devvm (CentOS/lib64 system)
  • Verified cmake configure completes without hanging
  • Verified cadence_runner binary is produced at cmake-out/backends/cadence/cadence_runner

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 9, 2026

🔗 Helpful Links

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

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

✅ No Failures

As of commit 42ab86f with merge base c385ed0 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 9, 2026
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Mar 9, 2026

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

- Detect lib vs lib64 install directory in build_cadence_runner.sh
- Add -DPYTHON_EXECUTABLE to prevent GLOB_RECURSE scanning entire filesystem
- Update generic/operators/CMakeLists.txt with actual source filenames
@aliafzal aliafzal force-pushed the fix-cadence-build branch from 8f199fb to 42ab86f Compare March 9, 2026 21:06
@aliafzal aliafzal added the release notes: cadence Changes to the Cadence backend delegate label Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: cadence Changes to the Cadence backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants