Skip to content

Conversation

eigen-k
Copy link
Contributor

@eigen-k eigen-k commented Aug 5, 2025

Summary: There is a user request to pass a custom PassManager to the EdgeProgramManager::transform(). This diff extends method API appropriately.

Differential Revision: D79292048

Copy link

pytorch-bot bot commented Aug 5, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 3 Unrelated Failures

As of commit 15c6cb4 with merge base b114f9c (image):

NEW FAILURES - The following jobs have failed:

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.

@meta-cla meta-cla 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 Aug 5, 2025
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 5, 2025
Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

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

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 5, 2025
Summary:
Pull Request resolved: pytorch#13140

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Differential Revision: D79292048
@eigen-k eigen-k force-pushed the export-D79292048 branch 2 times, most recently from 0c18261 to d095216 Compare August 6, 2025 18:36
eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

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

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

@eigen-k eigen-k added the release notes: none Do not include this in the release notes label Aug 6, 2025
eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

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

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
Summary:
Pull Request resolved: pytorch#13140

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

Differential Revision: D79292048
eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

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

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
…nager as a "passes" param variant. (pytorch#13140)

Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

Differential Revision: D79292048
eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

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

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

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

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request Aug 6, 2025
…nager as a "passes" param variant. (pytorch#13140)

Summary:
Pull Request resolved: pytorch#13140

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

Differential Revision: D79292048
…nager as a "passes" param variant. (pytorch#13140)

Summary:

There is a user request to pass a custom PassManager to the  EdgeProgramManager::transform(). This diff extends method API appropriately.

Reviewed By: larryliu0820

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

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

@eigen-k eigen-k changed the title Extend tranform() method params with PassManager. Extend EdgeProgramManager::transform() method declaration with PassManager as a "passes" param variant. Aug 7, 2025
@facebook-github-bot facebook-github-bot merged commit afdbb85 into pytorch:main Aug 7, 2025
98 of 104 checks passed
agrima1304 pushed a commit to agrima1304/executorch that referenced this pull request Aug 26, 2025
…nager as a "passes" param variant.

Differential Revision: D79292048

Pull Request resolved: pytorch#13140
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: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants