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

Reduce global memory reads #1731

Merged

charleskawczynski merged 1 commit into main from ck/min_reads

May 16, 2024

Member

charleskawczynski commented May 15, 2024

This PR reduces memory reads in our band_matrix_solve!, by reusing register data from previous loop iterations.

charleskawczynski added the performance label

charleskawczynski requested a review from dennisYatunin

May 15, 2024 21:19


          Reduce global memory reads

dabec5a

charleskawczynski force-pushed the ck/min_reads branch from 56b9203 to dabec5a Compare

May 16, 2024 12:57

Member Author

charleskawczynski commented May 16, 2024

I tried this out on the prognostic edmf on the A100, and ldiv! went from 291.34 ms to 271.5 ms (93% of what it used to cost).

charleskawczynski merged commit a9d80a4 into main

11 of 14 checks passed

charleskawczynski deleted the ck/min_reads branch

May 16, 2024 13:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment