Skip to content

[reland2][inductor] Add an AOT compilation mode for Inductor CPP backend#96520

Closed
desertfire wants to merge 2 commits intogh/desertfire/95/basefrom
gh/desertfire/95/head
Closed

[reland2][inductor] Add an AOT compilation mode for Inductor CPP backend#96520
desertfire wants to merge 2 commits intogh/desertfire/95/basefrom
gh/desertfire/95/head

Conversation

@desertfire
Copy link
Contributor

@desertfire desertfire commented Mar 10, 2023

Summary: This is a reland of #94822.
Solved the long compilation issue for inductor cpp tests.

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 10, 2023

🔗 Helpful Links

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

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

✅ No Failures

As of commit 59e54c7:
💚 Looks good so far! There are no failures yet. 💚

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

desertfire added a commit that referenced this pull request Mar 10, 2023
Summary: This is a reland of #94822.
Solved the long compilation issue for inductor cpp tests.

ghstack-source-id: 9f696b4
Pull Request resolved: #96520
@desertfire desertfire requested review from huydhn and malfet March 10, 2023 18:55
@desertfire
Copy link
Contributor Author

The execution time for this shard is back to normal now, https://github.com/pytorch/pytorch/actions/runs/4385847270/jobs/7679481242

@huydhn huydhn added the ciflow/trunk Trigger trunk jobs on your pull request label Mar 10, 2023
Copy link
Contributor

@huydhn huydhn left a comment

Choose a reason for hiding this comment

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

LGTM! I double check the TTS in the PR, and it looks normal inclusing ASAN and macos TTS

…or CPP backend"

Summary: This is a reland of #94822.
Solved the long compilation issue for inductor cpp tests.

cc soumith voznesenskym penguinwu anijain2305 EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng Xia-Weiwen wenzhe-nrv jiayisunx peterbell10

[ghstack-poisoned]
@desertfire
Copy link
Contributor Author

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: This PR needs a label
If your changes are user facing and intended to be a part of release notes, please use a label starting with release notes:.

If not, please add the topic: not user facing label.

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

Details for Dev Infra team Raised by workflow job

@desertfire desertfire added the topic: not user facing topic category label Mar 14, 2023
@desertfire
Copy link
Contributor Author

@pytorchbot merge

@pytorchmergebot
Copy link
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

cyyever pushed a commit to cyyever/pytorch_private that referenced this pull request Mar 23, 2023
…end (#96520)

Summary: This is a reland of pytorch/pytorch#94822.
Solved the long compilation issue for inductor cpp tests.

Pull Request resolved: pytorch/pytorch#96520
Approved by: https://github.com/huydhn, https://github.com/malfet
cyyever pushed a commit to cyyever/pytorch_private that referenced this pull request Mar 27, 2023
…end (#96520)

Summary: This is a reland of pytorch/pytorch#94822.
Solved the long compilation issue for inductor cpp tests.

Pull Request resolved: pytorch/pytorch#96520
Approved by: https://github.com/huydhn, https://github.com/malfet
@facebook-github-bot facebook-github-bot deleted the gh/desertfire/95/head branch June 8, 2023 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants