Skip to content

torchgen: support exception boundary for ExecuTorch functions#144341

Closed
swolchok wants to merge 5 commits into
gh/swolchok/714/basefrom
gh/swolchok/714/head
Closed

torchgen: support exception boundary for ExecuTorch functions#144341
swolchok wants to merge 5 commits into
gh/swolchok/714/basefrom
gh/swolchok/714/head

Conversation

@swolchok

@swolchok swolchok commented Jan 7, 2025

Copy link
Copy Markdown
Contributor

Stack from ghstack (oldest at bottom):

Needed for ExecuTorch diff D67904052.

Differential Revision: D67906411

Needed for ExecuTorch diff D67904052.

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

[ghstack-poisoned]
@pytorch-bot

pytorch-bot Bot commented Jan 7, 2025

Copy link
Copy Markdown

🔗 Helpful Links

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

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

✅ You can merge normally! (3 Unrelated Failures)

As of commit 4eeb6f3 with merge base 8cc6f17 (image):

BROKEN TRUNK - The following jobs failed but were 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.

@pytorch-bot pytorch-bot Bot added the topic: not user facing topic category label Jan 7, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

…ons"

Needed for ExecuTorch diff D67904052.

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

[ghstack-poisoned]
swolchok added a commit that referenced this pull request Jan 7, 2025
Pull Request resolved: #144341

Needed for ExecuTorch diff D67904052.

Differential Revision: [D67906411](https://our.internmc.facebook.com/intern/diff/D67906411/)
ghstack-source-id: 260530126
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

…ons"

Needed for ExecuTorch diff D67904052.

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

[ghstack-poisoned]
swolchok added a commit that referenced this pull request Jan 7, 2025
Pull Request resolved: #144341

Needed for ExecuTorch diff D67904052.
ghstack-source-id: 260556147

Differential Revision: [D67906411](https://our.internmc.facebook.com/intern/diff/D67906411/)
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

…ons"

Needed for ExecuTorch diff D67904052.

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

[ghstack-poisoned]
swolchok added a commit that referenced this pull request Jan 9, 2025
Pull Request resolved: #144341

Needed for ExecuTorch diff D67904052.
ghstack-source-id: 260813260

Differential Revision: [D67906411](https://our.internmc.facebook.com/intern/diff/D67906411/)
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@swolchok swolchok requested review from Jack-Khuu and larryliu0820 and removed request for larryliu0820 January 10, 2025 03:07
@Jack-Khuu

Copy link
Copy Markdown
Contributor

This passes the smell test, but it's been a while since I've looked at this code

cc: @larryliu0820 who is will be more familiar

@pytorch-bot pytorch-bot Bot added the ciflow/trunk Trigger trunk jobs on your pull request label Jan 15, 2025
…ons"

Needed for ExecuTorch diff D67904052.

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

[ghstack-poisoned]
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

swolchok added a commit that referenced this pull request Jan 30, 2025
Pull Request resolved: #144341

Needed for ExecuTorch diff D67904052.
ghstack-source-id: 263858638

Differential Revision: [D67906411](https://our.internmc.facebook.com/intern/diff/D67906411/)
@swolchok

Copy link
Copy Markdown
Contributor Author

@pytorchbot merge

@pytorchmergebot

Copy link
Copy Markdown
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@github-actions github-actions Bot deleted the gh/swolchok/714/head branch March 2, 2025 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request fb-exported Merged topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants