Skip to content

[NFC] [AutoDiff] Improve SILGen linear map thunking documentation. #26557

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

Merged
merged 2 commits into from
Aug 8, 2019

Conversation

dan-zheng
Copy link
Contributor

  • Add documentation comments clarifying differential/pullback thunk self reordering logic.
  • Change SILGenFunction::getThunkedAutoDiffLinearMap to use AutoDiffLinearMapKind.
  • Various gardening.

- Add documentation comments clarifying differential/pullback thunk self
  reordering logic.
- Change `SILGenFunction::getThunkedAutoDiffLinearMap` to use
  `AutoDiffLinearMapKind`.
- Various gardening.
@dan-zheng dan-zheng added the tensorflow This is for "tensorflow" branch PRs. label Aug 8, 2019
@dan-zheng dan-zheng requested a review from rxwei August 8, 2019 15:25
@dan-zheng
Copy link
Contributor Author

@swift-ci Please test tensorflow

Address review feedback.
Fix doc comment.
@dan-zheng
Copy link
Contributor Author

@swift-ci Please test tensorflow

@dan-zheng dan-zheng merged commit da03d99 into swiftlang:tensorflow Aug 8, 2019
@dan-zheng dan-zheng deleted the autodiff-silgen-thunks branch August 8, 2019 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tensorflow This is for "tensorflow" branch PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants