-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[reland][inductor] Add an AOT compilation mode for Inductor CPP backend #95985
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Summary: This is a reland of #94822 [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/95985
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Merge Blocking SEVsThere is 1 active merge blocking SEVs. Please view them below:
If you must merge, use ❌ 1 FailuresAs of commit ad96097: NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@pytorchbot rebase |
|
@pytorchbot successfully started a rebase job. Check the current status here |
|
Rebase failed due to Command |
…r CPP backend" Summary: This is a reland of #94822 cc soumith voznesenskym yanboliang penguinwu anijain2305 EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng Xia-Weiwen wenzhe-nrv jiayisunx peterbell10 [ghstack-poisoned]
|
@pytorchbot rebase |
|
@pytorchbot successfully started a rebase job. Check the current status here |
…r CPP backend" Summary: This is a reland of #94822 cc soumith voznesenskym yanboliang penguinwu anijain2305 EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng Xia-Weiwen wenzhe-nrv jiayisunx peterbell10 [ghstack-poisoned]
|
Successfully rebased |
|
@pytorchbot merge |
Merge startedYour 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 |
Merge failedReason: 1 mandatory check(s) failed. The first few are: Dig deeper by viewing the failures on hud |
|
@pytorchbot merge -f "macos test is flaky" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
|
@pytorchbot revert -m 'Sorry for reverting this. It increased the test time significantly for ASAN (and may be other test shards). ASAN tests on PR passed but it was barely not timing out. I have updated my initial findings in #96378' -c weird |
|
@pytorchbot successfully started a revert job. Check the current status here. |
|
@desertfire your PR has been successfully reverted. |
…PP backend (#95985)" This reverts commit deaf9e5. Reverted #95985 on behalf of https://github.com/huydhn due to Sorry for reverting this. It increased the test time significantly for ASAN (and may be other test shards). ASAN tests on PR passed but it was barely not timing out. I have updated my initial findings in #96378
|
Please note, that it also added about 2h of test time to MacOS x86 shards and about 30 min of test time to MacOS M1. Does this change really need such exhaustive testing? |
|
I think it only affected CPU tests. Let me look into it. |
…nd (#95985) Summary: This is a reland of pytorch/pytorch#94822 Pull Request resolved: pytorch/pytorch#95985 Approved by: https://github.com/jansel
…PP backend (#95985)" This reverts commit deaf9e5. Reverted pytorch/pytorch#95985 on behalf of https://github.com/huydhn due to Sorry for reverting this. It increased the test time significantly for ASAN (and may be other test shards). ASAN tests on PR passed but it was barely not timing out. I have updated my initial findings in pytorch/pytorch#96378
…nd (#95985) Summary: This is a reland of pytorch/pytorch#94822 Pull Request resolved: pytorch/pytorch#95985 Approved by: https://github.com/jansel
…PP backend (#95985)" This reverts commit deaf9e5. Reverted pytorch/pytorch#95985 on behalf of https://github.com/huydhn due to Sorry for reverting this. It increased the test time significantly for ASAN (and may be other test shards). ASAN tests on PR passed but it was barely not timing out. I have updated my initial findings in pytorch/pytorch#96378
…nd (pytorch#95985) Summary: This is a reland of pytorch#94822 Pull Request resolved: pytorch#95985 Approved by: https://github.com/jansel
…PP backend (pytorch#95985)" This reverts commit deaf9e5. Reverted pytorch#95985 on behalf of https://github.com/huydhn due to Sorry for reverting this. It increased the test time significantly for ASAN (and may be other test shards). ASAN tests on PR passed but it was barely not timing out. I have updated my initial findings in pytorch#96378
Stack from ghstack (oldest at bottom):
Summary: This is a reland of #94822
cc @soumith @voznesenskym @yanboliang @penguinwu @anijain2305 @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @Xia-Weiwen @wenzhe-nrv @jiayisunx @peterbell10