Skip to content

Fix Cadence CPU runner CMake build#18021

Merged
meta-codesync[bot] merged 1 commit intomainfrom
fix-cadence-build
Mar 10, 2026
Merged

Fix Cadence CPU runner CMake build#18021
meta-codesync[bot] merged 1 commit intomainfrom
fix-cadence-build

Conversation

@aliafzal
Copy link
Contributor

@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.

❌ 4 New Failures, 24 Pending

As of commit b3190ac with merge base 7b5c60d (image):

NEW FAILURES - The following jobs have failed:

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.

@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
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

Reviewed By: hsharma35, zonglinpeng

Differential Revision: D95846702

Pulled By: aliafzal
@meta-codesync meta-codesync bot force-pushed the fix-cadence-build branch from 42ab86f to b3190ac Compare March 10, 2026 16:57
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Mar 10, 2026

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

@meta-codesync meta-codesync bot merged commit c6b9d34 into main Mar 10, 2026
217 of 225 checks passed
@meta-codesync meta-codesync bot deleted the fix-cadence-build branch March 10, 2026 18:25
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. fb-exported meta-exported release notes: cadence Changes to the Cadence backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants