Skip to content

Conversation

manuelcandales
Copy link
Contributor

Differential Revision: D74765433

@manuelcandales manuelcandales requested a review from swolchok as a code owner May 20, 2025 15:34
Copy link

pytorch-bot bot commented May 20, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit ee3c602 with merge base 936ac2b (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-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 May 20, 2025
@facebook-github-bot
Copy link
Contributor

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

@manuelcandales manuelcandales added the release notes: ops & kernels Changes to the opset and any new / changed kernel implementations label May 20, 2025

} // namespace

Tensor& index_put_(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it worth making the out variant call the inplace one on out?

manuelcandales added a commit to manuelcandales/executorch-1 that referenced this pull request May 21, 2025
Summary: Pull Request resolved: pytorch#10996

Reviewed By: JacobSzwejbka

Differential Revision: D74765433
manuelcandales added a commit to manuelcandales/executorch-1 that referenced this pull request May 21, 2025
Summary: Pull Request resolved: pytorch#10996

Reviewed By: JacobSzwejbka

Differential Revision: D74765433
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

manuelcandales added a commit to manuelcandales/executorch-1 that referenced this pull request May 21, 2025
Summary: Pull Request resolved: pytorch#10996

Reviewed By: JacobSzwejbka

Differential Revision: D74765433
manuelcandales added a commit to manuelcandales/executorch-1 that referenced this pull request May 21, 2025
Summary: Pull Request resolved: pytorch#10996

Reviewed By: JacobSzwejbka

Differential Revision: D74765433
@facebook-github-bot
Copy link
Contributor

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

manuelcandales added a commit to manuelcandales/executorch-1 that referenced this pull request May 22, 2025
Summary: Pull Request resolved: pytorch#10996

Reviewed By: JacobSzwejbka

Differential Revision: D74765433
@facebook-github-bot
Copy link
Contributor

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

Summary: Pull Request resolved: pytorch#10996

Reviewed By: JacobSzwejbka

Differential Revision: D74765433
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot facebook-github-bot merged commit 62873b3 into pytorch:main May 22, 2025
88 of 90 checks passed
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 release notes: ops & kernels Changes to the opset and any new / changed kernel implementations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants