Skip to content

Dgemm loop unroll and 4x1, 4x2 dgemv VSX implementation for power10.#5453

Merged
martin-frbg merged 3 commits into
OpenMathLib:developfrom
pratiklp00:dgemm_optimization
Oct 28, 2025
Merged

Dgemm loop unroll and 4x1, 4x2 dgemv VSX implementation for power10.#5453
martin-frbg merged 3 commits into
OpenMathLib:developfrom
pratiklp00:dgemm_optimization

Conversation

@pratiklp00
Copy link
Copy Markdown
Contributor

This PR adds the loop unroll in dgemm_kernal_power10.c and VSX implementation of dgemv_kernel_4x2 and dgemv_kernel_4x1.

@pratiklp00 pratiklp00 marked this pull request as draft September 22, 2025 04:14
__builtin_mma_xvf64ger (&acc5, rowB1, rowA[2]);
__builtin_mma_xvf64ger (&acc6, rowB, rowA[3]);
__builtin_mma_xvf64ger (&acc7, rowB1, rowA[3]);
for (l = 1; l < temp; l++)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can we have a macro like this for https://github.com/OpenMathLib/OpenBLAS/blob/develop/kernel/power/sgemm_kernel_power10.c#L128 to reduce the number of lines?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes. I will add the macro same as sgemm.

@pratiklp00
Copy link
Copy Markdown
Contributor Author

Please review the changes.

@pratiklp00 pratiklp00 marked this pull request as ready for review October 14, 2025 05:10
@pratiklp00
Copy link
Copy Markdown
Contributor Author

This failure dosent seems to be caused by this PR.

@pratiklp00
Copy link
Copy Markdown
Contributor Author

Hi @martin-frbg could you please Review the PR?

@martin-frbg martin-frbg added this to the 0.3.31 milestone Oct 28, 2025
@martin-frbg martin-frbg merged commit 0c59ae0 into OpenMathLib:develop Oct 28, 2025
84 of 93 checks passed
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.

3 participants