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

Add dsp and mve support to transpose conv int8 #103

Merged

Conversation

ArmRyan
Copy link
Collaborator

@ArmRyan ArmRyan commented Feb 6, 2024

  • Adds new support functions to read and pad 2 int8s
  • Adds new support functions to allow addition to read and pad
  • Adds dsp optimizations for arm_nn_mat_mult_nt_t_s8_s32
  • Adds mve optimizations for arm_nn_mat_mult_nt_t_s8_s32
  • Adds mve requantization to arm_transpose_conv_s8
  • Adds new unit test

Change-Id: I0ebb09916a4a09a1299799831e6dd28cd9c650ed

@mansnils
Copy link
Contributor

mansnils commented Feb 9, 2024

Could you add arm_transpose_conv_get_buffer_sizes_s8.c to ARM.CMSIS-NN.pdsc? It was unfortunately missing even before this PR.

 * Adds new support functions to read and pad 2 int8s
 * Adds new support functions to allow addition to read and pad
 * Adds dsp optimizations for arm_nn_mat_mult_nt_t_s8_s32
 * Adds mve optimizations for arm_nn_mat_mult_nt_t_s8_s32
 * Adds mve requantization to arm_transpose_conv_s8
 * Adds new unit test

Change-Id: I0ebb09916a4a09a1299799831e6dd28cd9c650ed
Signed-off-by: Ryan O'Shea <ryan.oshea3@arm.com>
@ArmRyan ArmRyan force-pushed the experimental/int8-transpose-conv-dsp-mve branch from f524942 to 15db276 Compare February 9, 2024 12:07
 * update pdsc file with transpose_conv_get_buffer_sizes
 * rename read_pad_and_add to read_pad_and_add_s8

Change-Id: Ia2c160b580ef297e0bb506b18b21a4fb6d861c6a
Signed-off-by: Ryan O'Shea <ryan.oshea3@arm.com>
@ArmRyan
Copy link
Collaborator Author

ArmRyan commented Feb 9, 2024

added the buffer size file to pdsc :)

@mansnils mansnils merged commit 9eacdff into ARM-software:main Feb 12, 2024
1 of 2 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