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

Use calls to cblas_daxpy to calculate the derivative of functions with cblas_ddot #226

Merged
merged 1 commit into from
Jul 29, 2021

Conversation

reikdas
Copy link
Collaborator

@reikdas reikdas commented Jul 8, 2021

No description provided.

@reikdas reikdas requested a review from wsmoses July 8, 2021 17:09
@reikdas reikdas force-pushed the reikdas/ddot-adjoint branch 3 times, most recently from 75b7733 to 50ea0d2 Compare July 22, 2021 17:44
@reikdas reikdas requested a review from wsmoses July 22, 2021 17:45
@reikdas reikdas force-pushed the reikdas/ddot-adjoint branch 2 times, most recently from bdf9430 to 9c82ab0 Compare July 27, 2021 08:35
@reikdas reikdas requested a review from wsmoses July 27, 2021 08:47
@reikdas reikdas requested a review from wsmoses July 27, 2021 20:04
enzyme/Enzyme/Utils.cpp Outdated Show resolved Hide resolved
@wsmoses
Copy link
Member

wsmoses commented Jul 28, 2021 via email

@reikdas reikdas requested a review from wsmoses July 28, 2021 15:03
Copy link
Member

@wsmoses wsmoses left a comment

Choose a reason for hiding this comment

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

Excluding minor comments (see the lookup, and stride), LGTM.

Derivative of cblas_ddot can be calculated with calls to cblas_daxpy.
Similarly for cblas_sdot.
@reikdas reikdas merged commit ff17a4e into main Jul 29, 2021
@reikdas reikdas deleted the reikdas/ddot-adjoint branch July 29, 2021 05:26
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.

2 participants