Skip to content

Reuse the existing clone of the ios toolchain for Core ML.#3575

Closed
shoumikhin wants to merge 1 commit intopytorch:mainfrom
shoumikhin:export-D57225246
Closed

Reuse the existing clone of the ios toolchain for Core ML.#3575
shoumikhin wants to merge 1 commit intopytorch:mainfrom
shoumikhin:export-D57225246

Conversation

@shoumikhin
Copy link
Contributor

Summary: .

Differential Revision: D57225246

@pytorch-bot
Copy link

pytorch-bot bot commented May 10, 2024

🔗 Helpful Links

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

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

❌ 3 New Failures

As of commit 47ccfd8 with merge base 87d828a (image):

NEW FAILURES - The following jobs have failed:

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 10, 2024
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

@mergennachin
Copy link
Contributor

@cymbalrush - please take a look. we recently added dependency to ios-toolchain in the third-party directory. This allows us to just reuse that instead.

@cymbalrush
Copy link
Contributor

Looks good, thank you!

Summary:

.

Reviewed By: mergennachin

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 4b5e434.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants