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.

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

[Autodiff] Adds logic to generate specialized functions in the closure-spec pass #73596

Merged
merged 3 commits into from
May 15, 2024

Conversation

jkshtj
Copy link
Collaborator

@jkshtj jkshtj commented May 13, 2024

Approved in this PR.

PR to merge into Swift main branch.

@jkshtj jkshtj requested a review from eeckstein as a code owner May 13, 2024 16:36
@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@swift-ci please test

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@eeckstein could I get a quick approval on this?

Copy link
Member

@eeckstein eeckstein left a comment

Choose a reason for hiding this comment

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

lgtm!

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@swift-ci please test

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@swift-ci please test macos

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@swift-ci please test

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@shahmishal swift-ci seems to be broken.

@shahmishal
Copy link
Member

@jkshtj Do you have commit access to swift repo?

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@jkshtj Do you have commit access to swift repo?

I do

@shahmishal
Copy link
Member

shahmishal commented May 13, 2024

I see the trigger comment has a extra space:

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@swift-ci please test

@shahmishal
Copy link
Member

Fixed the issue with Windows trigger by adding you to the allow list.

@shahmishal
Copy link
Member

@swift-ci test Windows

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

I see the trigger comment has a extra space:

Ah, I'm sorry for the mistake. Looks like that was it.

Is there a way to make these swift-ci invocations agnostic to certain cosmetic things - capitalization, spaces etc.?

Addresses some other surfacial feedback as well.
@jkshtj
Copy link
Collaborator Author

jkshtj commented May 13, 2024

@swift-ci please test

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 14, 2024

@swift-ci please test linux

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 14, 2024

@swift-ci please test

@jkshtj
Copy link
Collaborator Author

jkshtj commented May 14, 2024

@swift-ci test Windows

@jkshtj jkshtj merged commit c37d31c into apple:main May 15, 2024
5 checks passed
@jkshtj jkshtj deleted the specialize branch May 15, 2024 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants