Skip to content

Conversation

perheld
Copy link
Collaborator

@perheld perheld commented Jun 13, 2025

The runtime is written in C++ and had a bunch of cstyle castings and mixed operator tokens. This commit addresses these warnings.

Signed-off-by: per.held@arm.com

Change-Id: I8fa9d36ce61c98896b56b2db920d9d2db9fe972e

cc @larryliu0820 @JacobSzwejbka @lucylq @digantdesai @freddan80 @per @zingo @oscarandersson8218

@perheld perheld requested a review from digantdesai as a code owner June 13, 2025 11:10
@perheld perheld added module: runtime Issues related to the core runtime and code under runtime/ partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk labels Jun 13, 2025
Copy link

pytorch-bot bot commented Jun 13, 2025

🔗 Helpful Links

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

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

❌ 1 New Failure, 2 Pending, 3 Unrelated Failures

As of commit 34685d4 with merge base d9b3cf4 (image):

NEW FAILURE - The following job has failed:

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

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

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

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

@facebook-github-bot facebook-github-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 Jun 13, 2025
The runtime is written in C++ and had a bunch of cstyle castings and
mixed operator tokens. This commit addresses these warnings.

Signed-off-by: per.held@arm.com

Change-Id: I8fa9d36ce61c98896b56b2db920d9d2db9fe972e
@perheld perheld force-pushed the upstream/ph-fix-cpp-runtime branch from 6299f4c to 34685d4 Compare June 13, 2025 11:11
@perheld perheld added the release notes: arm Changes to the ARM backend delegate label Jun 13, 2025
@zingo
Copy link
Collaborator

zingo commented Jun 13, 2025

Tests broken on main also

@zingo zingo merged commit e58fdb6 into pytorch:main Jun 13, 2025
189 of 193 checks passed
@perheld perheld deleted the upstream/ph-fix-cpp-runtime branch June 13, 2025 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: runtime Issues related to the core runtime and code under runtime/ partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: arm Changes to the ARM backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants