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

Rather than using the default GEMM algorithm, use KK_DENSE algorithm. #343

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ddement
Copy link
Collaborator

@ddement ddement commented Jan 24, 2025

The result is substantial performance improvements (10-30% of the total runtime) for the Floating Platform, IEA15 Rotor, and IEA15 Rotor with 3000 elements on CPU. Similar performance improvements are seen on GPU, with the exception of the IEA15 with 3000 elements. However, the performance of that test is dominated by the system solve, so the result is no net change in performance.

The result is substantial performance improvements (10-30%) for the Floating Platform,
IEA15 Rotor, and IEA15 Rotor with 3000 elements on CPU.  Similar performance
improvements are seen on GPU, with the exception of the IEA15 with 3000 elements.
However, the performance of that test is dominated by the system solve, so the result
is no net change in performance.
@ddement ddement self-assigned this Jan 24, 2025
@ddement ddement marked this pull request as ready for review January 24, 2025 23:05
@ddement ddement changed the title Rather than using the default algorithm, use KK_DENSE algorithm. Rather than using the default GEMM algorithm, use KK_DENSE algorithm. Jan 24, 2025
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.

1 participant