Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DO NOT MERGE] Why is enzyme frontend PR failing in post-opt-dist crashes test #131685

Closed
wants to merge 3 commits into from
Closed

[DO NOT MERGE] Why is enzyme frontend PR failing in post-opt-dist crashes test #131685

wants to merge 3 commits into from

Conversation

jieyouxu
Copy link
Member

@jieyouxu jieyouxu commented Oct 14, 2024 •
edited
Loading

r? @ghost

try-job: dist-x86_64-msvc

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 14, 2024
@jieyouxu jieyouxu marked this pull request as draft October 14, 2024 13:35
@jieyouxu jieyouxu removed A-testsuite Area: The testsuite used to check the correctness of rustc T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 14, 2024
@jieyouxu jieyouxu mentioned this pull request Oct 14, 2024
Merged
@jieyouxu
Copy link
Member Author

Let's see if this takes forever on default linux try job.
@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 14, 2024
@bors
Auto merge of rust-lang#131685 - jieyouxu:exp-enzyme-frontend, r=<try>
a938246 
[DO NOT MERGE] Why is enzyme frontend PR failing in post-opt-dist crashes test

r? `@ghost`
@bors
Copy link
Contributor

bors commented Oct 14, 2024

⌛ Trying commit faa347a with merge a938246...

@bors bors mentioned this pull request Oct 14, 2024
Merged
@bors
Copy link
Contributor

bors commented Oct 14, 2024

☀️ Try build successful - checks-actions
Build commit: a938246 (a938246ff28ff9d6b3bcb0f2982bcf44a142691a)

@jieyouxu
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 14, 2024
@bors
Auto merge of rust-lang#131685 - jieyouxu:exp-enzyme-frontend, r=<try>
a03d61d 
[DO NOT MERGE] Why is enzyme frontend PR failing in post-opt-dist crashes test

r? `@ghost`

try-job: dist-x86_64-msvc
@bors
Copy link
Contributor

bors commented Oct 14, 2024

⌛ Trying commit faa347a with merge a03d61d...

@bors
Copy link
Contributor

bors commented Oct 14, 2024

☀️ Try build successful - checks-actions
Build commit: a03d61d (a03d61d912c930f6a661fd8a10667291e1fd5998)

@jieyouxu
Copy link
Member Author

???

@jieyouxu
Copy link
Member Author

Apparently msvc didn't like the PR yesterday.

@jieyouxu jieyouxu closed this Oct 15, 2024
@jieyouxu jieyouxu deleted the exp-enzyme-frontend branch October 15, 2024 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jieyouxu @bors @rustbot @ZuseZ4