Skip to content

Add c10::irange to ExecuTorch #8572

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 19, 2025
Merged

Add c10::irange to ExecuTorch #8572

merged 1 commit into from
Feb 19, 2025

Conversation

pytorchbot
Copy link
Collaborator

This PR was created by the merge bot to help merge the original PR into the main branch.
ghstack PR number: #8554 by @swolchok
^ Please use this as the source of truth for the PR details, comments, and reviews
ghstack PR base: https://github.com/pytorch/executorch/tree/gh/swolchok/270/base
ghstack PR head: https://github.com/pytorch/executorch/tree/gh/swolchok/270/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/main
Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/swolchok/270/orig
@diff-train-skip-merge

Pull Request resolved: #8554

irange is a header-only utility that is both more readable than the usual way to write for loops and also solves -Wsign-compare issues. I've previously vetted that it generates the same assembly as a regular for loop.
ghstack-source-id: 267194238
@exported-using-ghexport

Differential Revision: [D69817195](https://our.internmc.facebook.com/intern/diff/D69817195/)
Copy link

pytorch-bot bot commented Feb 19, 2025

🔗 Helpful Links

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

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

❌ 6 New Failures, 6 Pending, 1 Unrelated Failure

As of commit 332851e with merge base cc3974f (image):

NEW FAILURES - The following jobs have failed:

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

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 Feb 19, 2025
@swolchok swolchok added the release notes: api Changes to public facing apis (any interfaces, pybinded runtime methods, etc.) label Feb 19, 2025
@swolchok swolchok merged commit 0d3cbe8 into main Feb 19, 2025
41 of 49 checks passed
@swolchok swolchok deleted the gh/swolchok/270/orig branch February 19, 2025 18:08
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: api Changes to public facing apis (any interfaces, pybinded runtime methods, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants