2019-05-30 09:18:42

by Vaneet Narang

[permalink] [raw]
Subject: RE: [PATCH 2/2] zstd: use U16 data type for rankPos

[Reminder] Any Comments?

>rankPos structure variables value can not be more than 512.
>So it can easily be declared as U16 rather than U32.
 
>It will reduce stack usage of HUF_sort from 256 bytes to 128 bytes
 
>original:
>e24ddc01        sub     sp, sp, #256    ; 0x100
 
>changed:
>e24dd080        sub     sp, sp, #128    ; 0x80
 
Regards,
Vaneet Narang