Re: [PATCH 2/2] zstd: use U16 data type for rankPos

2019-06-06 Thread Nick Terrell
> On May 9, 2019, at 11:13 PM, Maninder Singh wrote: > > 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: > e92ddbf0push{r4, r5, r6,

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

2019-05-30 Thread Vaneet Narang
[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: >e24ddc01sub sp, sp, #256; 0x100   >changed: >e24dd080