Skip to content

Conversation

@Ryo-not-rio
Copy link
Contributor

@Ryo-not-rio Ryo-not-rio commented Nov 26, 2025

  • add brgemm optimizations

Copy link
Contributor

@nSircombe nSircombe left a comment

Choose a reason for hiding this comment

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

Could you add a line to the 'added' subheading in the [unreleased] section of the CHANGELOG.md

@Ryo-not-rio Ryo-not-rio force-pushed the add-conv branch 3 times, most recently from 2051452 to 1a0b556 Compare November 26, 2025 16:08
apply-github-patch uxlfoundation/oneDNN 8bdff1a2a6625432701363185a9bd34f7c22f241
# https://github.com/uxlfoundation/oneDNN/pull/4377 - cpu: aarch64: conv: optimize brgemm
apply-github-patch uxlfoundation/oneDNN 93b8cc29afc6ee9c2856436a5f5b10d5f1f2f2f1

Copy link
Contributor

Choose a reason for hiding this comment

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

Admittedly I'm being a bit pedantic here but is there any chance we could remove this trailing blank line? Then I'm happy to approve

Copy link
Contributor

@nSircombe nSircombe left a comment

Choose a reason for hiding this comment

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

looks good to me, thanks!

@Ryo-not-rio Ryo-not-rio merged commit 2a40e34 into main Dec 2, 2025
6 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.

4 participants