Skip to content

Support Span to construct from a single element similar to ArrayRef - #11757

Merged
facebook-github-bot merged 4 commits into
gh/cccclai/28/basefrom
gh/cccclai/28/head
Jun 18, 2025
Merged

Support Span to construct from a single element similar to ArrayRef#11757
facebook-github-bot merged 4 commits into
gh/cccclai/28/basefrom
gh/cccclai/28/head

Conversation

@cccclai

@cccclaicccclai commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

Stack from ghstack (oldest at bottom):

Copy the constructor from ArrayRef here

/// Construct a ArrayRef from a single element. Implicitly convert element
/// type. It is aligned with PyTorch's c10::ArrayRef.
/* implicit */constexprArrayRef(const T& OneElt)
: Data(&OneElt), Length(1) {}

so it's easier to use
@exported-using-ghexport

Differential Revision: D76825662

Differential Revision: D76825662

@pytorch-bot

pytorch-botBot commented Jun 17, 2025

Copy link
Copy Markdown

🔗 Helpful Links

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

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

✅ No Failures

As of commit 9405f04 with merge base 1309849 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-botfacebook-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 17, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D76825662

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D76825662

cccclai added a commit that referenced this pull request Jun 17, 2025
Pull Request resolved: #11757
Copy the constructor from ArrayRef here
https://github.com/pytorch/executorch/blob/5365c5559accc7a0d522eacdfb7385ce8914ef53/runtime/core/array_ref.h#L78-L81
so it's easier to use
ghstack-source-id: 291080862
@exported-using-ghexport
Differential Revision: [D76825662](https://our.internmc.facebook.com/intern/diff/D76825662/)
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D76825662

cccclai added a commit that referenced this pull request Jun 17, 2025
Pull Request resolved: #11757
Copy the constructor from ArrayRef here
https://github.com/pytorch/executorch/blob/5365c5559accc7a0d522eacdfb7385ce8914ef53/runtime/core/array_ref.h#L78-L81
so it's easier to use
ghstack-source-id: 291106564
@exported-using-ghexport
Differential Revision: [D76825662](https://our.internmc.facebook.com/intern/diff/D76825662/)
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D76825662

@swolchokswolchok added the release notes: api Changes to public facing apis (any interfaces, pybinded runtime methods, etc.) label Jun 17, 2025
@facebook-github-bot
facebook-github-bot merged commit b4edff6 into gh/cccclai/28/baseJun 18, 2025
@facebook-github-bot
facebook-github-bot deleted the gh/cccclai/28/head branch June 18, 2025 03:21
cccclai added a commit that referenced this pull request Jun 18, 2025
…11778)
This PR was created by the merge bot to help merge the original PR into
the main branch.
ghstack PR number: #11757 by
@cccclai
^ 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/cccclai/28/base
ghstack PR head:
https://github.com/pytorch/executorch/tree/gh/cccclai/28/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/main
Merge bot PR head:
https://github.com/pytorch/executorch/tree/gh/cccclai/28/orig
@diff-train-skip-merge
Co-authored-by: Chen Lai <chenlai@fb.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.fb-exportedrelease notes: apiChanges to public facing apis (any interfaces, pybinded runtime methods, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@cccclai@facebook-github-bot@JacobSzwejbka@swolchok