Skip to content

Refactor AVX tq functions to remove duplicate code#107

Merged
kpchoi merged 1 commit intomainfrom
avx_code_refactoring
Jun 30, 2025
Merged

Refactor AVX tq functions to remove duplicate code#107
kpchoi merged 1 commit intomainfrom
avx_code_refactoring

Conversation

@subhrajitm20
Copy link
Collaborator

  • Used macros to remove duplicate & clean up code
  • The output is bit-exact

Signed-off-by: subhrajitm20 <2003subhrajit@gmail.com>
@subhrajitm20 subhrajitm20 requested review from kpchoi and mss-park June 18, 2025 09:36
@kpchoi
Copy link
Collaborator

kpchoi commented Jun 22, 2025

For me, It is difficult to understand why this code refactoring is required and why it can lead an improvment.

Copy link
Collaborator

@kpchoi kpchoi left a comment

Choose a reason for hiding this comment

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

LGTM

@kpchoi kpchoi merged commit 4a95a7b into main Jun 30, 2025
5 checks passed
alejandro-arango-epicgames pushed a commit to alejandro-arango-epicgames/openapv that referenced this pull request Sep 29, 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.

2 participants